-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add PySB testcases #27
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good. Would test further before merging.
- _measurements.tsv | ||
observable_files: | ||
- _observables.tsv | ||
sbml_files: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shall we change that to pysb_files
directly?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
probably makes sense.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would leave as is for now, so the yaml is still syntactically correct for the current PEtab. Definitely needs changing in the future, but it needs to be discussed how that would look like, but I would avoid changing things twice.
@@ -9,7 +9,7 @@ | |||
|
|||
# problem -------------------------------------------------------------------- | |||
|
|||
model = DEFAULT_MODEL_FILE | |||
model = DEFAULT_SBML_FILE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unused everywhere... really needs some cleanup 😬 (elsewhere)
so we can reuse the observable formulas without modification
This reverts commit d10d55d.
'F821 undefined name' due to use of auto-export
Just copies of the original, as the alternative sbml model does not make sense here
Adds (experimental) PySB-PEtab versions of the test cases. Cases with Species IDs in the condition table are incompatible with PySB. They are still included here, and must fail. The model file is still provided under |
@FFroehlich : Can't request your review - good to go? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exact form of pysb support needs to be discussed with all, but fine to keep it as is here for the moment I guess.
@@ -0,0 +1 @@ | |||
. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
symlink cases/sbml
-> cases/
instead of moving the cases directories to cases/sbml
and breaking backward compatibility ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ahh. Yes, I thought this was supposed to be somewhere, but github shows this weirdly.
No description provided.