Open msevestre opened 1 year ago
I think it does not make sense indeed; the entry for human should be removed from tab_container_parameter_values
currently, distribution "wins" over the const value due to https://github.com/Open-Systems-Pharmacology/PK-Sim/blob/4dda6236eb76e2f8ed2208584acf52f73ec91995/src/PKSim.Core/Services/ParameterContainerTask.cs#L183-L185
Could make sense if e.g. a parameter is defined as distributed just for 1 population and as a const value for all other populations. But here it's defined for ALL populations, so the const value does not make any sense.
Yes it wins but I am wondering if we should remove it completely? Feels more like a db issue to me. This value is never used. Do we have parameters that are distributed for one population and constant for another?
Example: Gastric emptying time
and
We have code that removes one to add the other but....I can't help but feel we should not have a constant parameter in the first place. Or what am I missing? It's only distributed for a sub section of the populations? Weird