-
Notifications
You must be signed in to change notification settings - Fork 12
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
Allow using observables in noiseFormula #543
Comments
There seems to be something around that maybe could be used / adapted? |
This doesn't really require much adaptation. It's mostly about making a decision whether we want to allow that. (I think we should.) |
Following up on #543 and the discussion during the last PEtab editor meeting: There was general consent to allow using observableIDs in the `noiseFormula` column in the observables table. What was not discussed explicitly: * Should all observable IDs be allowed, or only the current one? * If so, should it be allowed to use observable IDs inside `observableFormula`? CC @PEtab-dev/petab-editors Closes ##543
Currently it is not allowed to use observable IDs in
noiseFormula
in the observables table (#368). It would be great to allow that.This would convenient for specifying relative errors. Although the
observableFormula
could be copied to thenoiseFormula
, it would be more readable and less error prone if one could simply use the respectiveobservableId
.Also, some tools do not support state variables, but only observables in noise models, so this would be convenient there too.
Opinions?
The text was updated successfully, but these errors were encountered: