-
Notifications
You must be signed in to change notification settings - Fork 30
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
Workaround for PEtab problems with state-dependent noise models #1791
Conversation
PEtab does currently not allow observables in noiseFormula and AMICI cannot handle states in sigma expressions. Therefore, where possible, replace species occurring in the error model definition by observableIds.
Codecov Report
@@ Coverage Diff @@
## develop #1791 +/- ##
===========================================
+ Coverage 72.95% 73.00% +0.05%
===========================================
Files 71 71
Lines 12237 12238 +1
===========================================
+ Hits 8927 8934 +7
+ Misses 3310 3304 -6
Flags with carried forward coverage won't be shown. Click here to find out more.
|
de392f4
to
b6167e5
Compare
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.
test with Raia_CancerResearch2011
bechmark model? if gradients don't match for that model just ignore?
|
|
In its current form, that one uses observables in |
Kudos, SonarCloud Quality Gate passed! |
PEtab does currently not allow observables in noiseFormula and AMICI cannot handle states in sigma expressions. Therefore, where possible, replace species occurring in the error model definition by observableIds.
See also #1788