-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dev to master #74
Merged
Merged
Dev to master #74
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
* Deterministic Safe Interface Code written * Creating some tests * Interface Safemode unit tests complete * Updated py_simulate_model to allow Safe Deterministic simulations Co-authored-by: William Poole <wpoole@caltech.edu>
* renamed local parameters extended to bioscrape propensity annotations * Added Unit Tests for SBML reading and writing with and without annotations * Model comparison removed from test_utils * changing paths to work across platforms * added pandas to requirements Co-authored-by: William Poole <wpoole@caltech.edu>
* Fixed Bioscrape XML loading and added depricated warning for bioscrape XML * Added tests to compare general propensities to bioscrape propensities Co-authored-by: William Poole <wpoole@caltech.edu>
* Clears c_vectors in model._create_vectors * Adds a test to make sure running initialize twice isn't a problem. Co-authored-by: William Poole <wpoole@caltech.edu>
ayush9pandey
approved these changes
Oct 13, 2020
ayush9pandey
added a commit
that referenced
this pull request
Nov 3, 2020
* Dev (#70) * Ode safe mode (#67) * Deterministic Safe Interface Code written * Creating some tests * Interface Safemode unit tests complete * Updated py_simulate_model to allow Safe Deterministic simulations * Bioscrape annotations fix (#65) * renamed local parameters extended to bioscrape propensity annotations * Added Unit Tests for SBML reading and writing with and without annotations * Model comparison removed from test_utils * changing paths to work across platforms * added pandas to requirements * Remove underscore from parameter definition (#68) * Fixed Bioscrape XML loading and added depricated warning for bioscrape XML * Added tests to compare general propensities to bioscrape propensities * C vector bug fix (#69): Clears c_vectors in model._create_vectors * Adds a test to make sure running initialize twice isn't a problem. Co-authored-by: William Poole <wpoole@caltech.edu> * Lineage beta (#71) * Fixed global volume distribution problem wasn't checking cell death correctly * Minor changes to lineages Fixed a bug where arrays of length 0 were being made. Tried to find other places to debug this seg-fault problem. I believe it might now be an issue when running bioscrape inside jupyter notebooks??? When running on the prompt I can run tons of simulations no problem. * Added new lineage examples * forgot the sbml file! Co-authored-by: William Poole <wpoole@caltech.edu> * Dev to master (#74) * Ode safe mode (#67) * Deterministic Safe Interface Code written * Creating some tests * Interface Safemode unit tests complete * Updated py_simulate_model to allow Safe Deterministic simulations Co-authored-by: William Poole <wpoole@caltech.edu> * Bioscrape annotations fix (#65) * renamed local parameters extended to bioscrape propensity annotations * Added Unit Tests for SBML reading and writing with and without annotations * Model comparison removed from test_utils * changing paths to work across platforms * added pandas to requirements Co-authored-by: William Poole <wpoole@caltech.edu> * Remove underscore from parameter definition (#68) * Fixed Bioscrape XML loading and added depricated warning for bioscrape XML * Added tests to compare general propensities to bioscrape propensities Co-authored-by: William Poole <wpoole@caltech.edu> * C vector bug fix (#69) * Clears c_vectors in model._create_vectors * Adds a test to make sure running initialize twice isn't a problem. Co-authored-by: William Poole <wpoole@caltech.edu> Co-authored-by: William Poole <wpoole@caltech.edu> * update readme travis tag (#75) Co-authored-by: William Poole <wp_ix@hotmail.com> Co-authored-by: William Poole <wpoole@caltech.edu>
WilliamIX
added a commit
that referenced
this pull request
Jan 6, 2021
* Ode safe mode (#67) * Deterministic Safe Interface Code written * Creating some tests * Interface Safemode unit tests complete * Updated py_simulate_model to allow Safe Deterministic simulations Co-authored-by: William Poole <wpoole@caltech.edu> * Bioscrape annotations fix (#65) * renamed local parameters extended to bioscrape propensity annotations * Added Unit Tests for SBML reading and writing with and without annotations * Model comparison removed from test_utils * changing paths to work across platforms * added pandas to requirements Co-authored-by: William Poole <wpoole@caltech.edu> * Remove underscore from parameter definition (#68) * Fixed Bioscrape XML loading and added depricated warning for bioscrape XML * Added tests to compare general propensities to bioscrape propensities Co-authored-by: William Poole <wpoole@caltech.edu> * C vector bug fix (#69) * Clears c_vectors in model._create_vectors * Adds a test to make sure running initialize twice isn't a problem. Co-authored-by: William Poole <wpoole@caltech.edu> Co-authored-by: William Poole <wpoole@caltech.edu>
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.
@ayush9pandey thanks for pointing out that dev was not fully merged into master. We should absolutely make sure these are merged [I thought I did this two weeks ago, but apparently it didn't go through completely somehow...]
Let me know if you have any comments on this PR - it should probably be merged before the Inference one.