AbstractDynodeRunner
code reorg and naming revamp
#316
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.
this PR does the following:
save_inference_timeseries
now takes particle tuples as a list instead of randomly selecting, allowing for better user control over its behavior. The method will also return the saved dataframe instead of a str of where it was saved to. If user passesNone
tosave_filename
the csv will not be saved at all, in case that is required.AbstractDynodeRunner
out intoutils
so they may be more easily referenced by other code, as well as declutter the class itself.timelines
totimeseries
when referring to the saving of metrics like vaccination or infection counts of a simulation.azure_visualizer_timelines.csv
auto-generated file tosimulation_timeseries.csv
so it is easier to understand what it contains. Also the csv is used in much more than just the shiny app visualizers...CLOSES #311
while this PR closes #311 some lasting software design decisions have made the request for parallelized loading of posterior particles not possible without substantial changes. I will create an issue #315 to track that