-
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
Add support for time-point specific overrides to petab.parameter_mapping #125
Comments
Wouldn't it be better to avoid the generation of additional conditions and instead duplicate the observables? This would also avoid additional simulations. |
Avoiding additional simulations is a good point. Besides that I don't see any (dis)advantages of the one over the other. |
(EDIT: no, this would be separate conditions anyways. so no problem there) |
is there any example in the benchmark having this feature at the moment? |
I'd propose to do that as a preprocessing step independently of the parameter mapping functions, creating a new set of tables. For the replicate-observable case: Something like This would involve:
|
I think so, but don't remember which one. should be simple to construct though. |
There should be several examples, e.g. the Isensee model. The control condition is use in many experiments with identical time points, but the observable parameters differ. |
Just need a short clarification @dweindl how to handle the case (if considered):
should it be also considered as a new replicate (e.g. |
Yes, it should. |
I came up with a solution, it works for the simple test case, but I am afraid it will not support all possible models/scenarios... Could you still @dweindl have a look? Maybe you can give me some guidance 😄 |
If it works for the test case it should be fine. I thought I considered all relevant scenarios. |
petab.parameter_mapping
does not support time-point specific parameter overrides. To work with the current mapping system, these measurements should be split into groups by overrides and returning them as separate conditions.The text was updated successfully, but these errors were encountered: