Issue 88 and 86: Update naming to be more consistent and add new overarching AbstractModel #89
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As the title this PR closes #88 and #86.
It:
generate_latent_process
->generate_latent
to match the pattern of the other modules.RandomWalkLatentProcess
->RandomWalk
again to match the pattern from the other module.make_epi_inference_model
->make_epi_aware
again for naming consistency and as discussed (this change lacks a connected issue but I think is in scope of this PR.epimodel
toepi_model
again for consistency.AbstractModel
scan
to expect a callable of typeAbstractModel
Note as this change touches a lot of files it may cause issues for #85 - sorry!