Skip to content
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

Merged
merged 16 commits into from
Nov 17, 2020
Merged

Add PySB testcases #27

merged 16 commits into from
Nov 17, 2020

Conversation

FFroehlich
Copy link
Collaborator

No description provided.

Copy link
Member

@dweindl dweindl left a 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:
Copy link
Member

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?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

probably makes sense.

Copy link
Member

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
Copy link
Member

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)

cases/0010/_model.xml Show resolved Hide resolved
dweindl and others added 4 commits July 31, 2020 14:05
@dweindl dweindl mentioned this pull request Nov 9, 2020
1 task
@dweindl
Copy link
Member

dweindl commented Nov 10, 2020

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 sbml_files: in the yaml file. This may change later on, depending on PEtab-dev/PEtab#436.

@dweindl
Copy link
Member

dweindl commented Nov 10, 2020

@FFroehlich : Can't request your review - good to go?

Copy link
Member

@yannikschaelte yannikschaelte left a 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 @@
.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?

Copy link
Member

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 ;)

Copy link
Member

@yannikschaelte yannikschaelte Nov 16, 2020

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.

@dweindl dweindl merged commit b22acb9 into PEtab-dev:master Nov 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants