History.record edited to improve Euler method simulation efficiency #290
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Changes the `record` class method in the History class such that it records the datasets dataframes as entries in a dictionary with the `time_stamp` as keys that map to corresponding dataframe for each element. The `self.dicts` property is introduced to account for this - this is a nested dictionary with the `eptm.elements` as keys and the above dictionaries as the values. This allows for appending the dataframes at each time step without having to load onto memory the increasingly large `self.datasets[element]` dataframes. The `update_datasets` class method is introduced in order to concatenate all the dataframes within `self.datasets[element]` at the end of the simulation. This will be accompanied by changes to the `EulerSolver` class method `solve` to run `History.update_datasets` athe the final time point.
Includes the `History.update_datasets` class method so that it runs at the last iteration of the `EulerSolver.solve` class method to generate the dataframes for History.datasets
edit record method to make simulations faster
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Changes the record class method in the History class such that it records the
self.datasets[element]dataframes as entries in a dictionary with thetime_stampas keys that map to corresponding dataframe for each element.The
self.dictsproperty is introduced to account for this - this is a nested dictionary with theeptm.elementsas keys and the above dictionaries as the values.This allows for appending the dataframes at each time step without having to load onto memory the increasingly large
self.datasets[element]dataframes.The
update_datasetsclass method is introduced in order to concatenate all the dataframes withinself.datasets[element]at the end of the simulation.This will be accompanied by changes to the
EulerSolverclass method solve to runHistory.update_datasetsat the final time point.