Improve Euler method simulation efficiency #292
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
Update history.py
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
Historyclass such that it records theself.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_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.solveclass method to run History.update_datasets at the final time point.