-
Notifications
You must be signed in to change notification settings - Fork 50
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
Cat cattle 2 #1960
Cat cattle 2 #1960
Conversation
@@ -67,7 +67,7 @@ INSERT INTO tab_species VALUES('Mouse','Mouse','Mouse',7,0,'Mouse',0); | |||
INSERT INTO tab_species VALUES('Rabbit','Rabbit','Rabbit',10,0,'Rabbit',0); | |||
INSERT INTO tab_species VALUES('Rat','Rat','Rat',6,0,'Rat',0); | |||
INSERT INTO tab_species VALUES('Cat','Cat','Cat',11,0,'Cat',0); | |||
INSERT INTO tab_species VALUES('Cattle','Cattle','Cattle',12,0,'Cattle',0); | |||
INSERT INTO tab_species VALUES('Cattle','Cattle (IV only)','Cattle',12,0,'Cattle',0); |
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.
Changed the display name, so that it's immediately clear that we cannot use cattle with oral applications
@@ -4482,7 +4521,7 @@ INSERT INTO tab_calculation_method_rates VALUES('VascularEndoVolume','VOLUME_Vas | |||
INSERT INTO tab_calculation_method_rates VALUES('VascularEndoVolume','VOLUME_VascularEndo_tes','SA_pls_int*d_endo','Volume'); | |||
INSERT INTO tab_calculation_method_rates VALUES('VascularEndoVolume','VOLUME_VascularEndo_uil','SA_pls_int*d_endo','Volume'); | |||
INSERT INTO tab_calculation_method_rates VALUES('VascularEndoVolume','VOLUME_VascularEndo_uje','SA_pls_int*d_endo','Volume'); | |||
INSERT INTO tab_calculation_method_rates VALUES('Lumen_PKSim','PARAM_EffectiveLumenConcentration','EnableSupersaturarion ? M/Liquid : min(Solubility/MW ; M/Liquid)','Concentration (molar)'); | |||
INSERT INTO tab_calculation_method_rates VALUES('Lumen_PKSim','PARAM_EffectiveLumenConcentration','Liquid>0 ? (EnableSupersaturarion ? M/Liquid : min(Solubility/MW ; M/Liquid)) : 0','Concentration (molar)'); |
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.
Changed the formula so that it does not return NaN if Liquid
is zero.
Otherwise it crashed with cattle, because the Gi-TRact is de-facto disabled there (all volumes are set to 0), and our new parameter generation in core did not create this parameter in the simulation at all, because it was NaN
if (catParameter.ValueOrigin.Description != "Copied from Beagle") | ||
continue; | ||
|
||
catParameter.Value.ShouldBeEqualTo(allConstBeagleParameter[i].Value, 1e-5); |
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.
Checking that if value origin says "Copied from Beagle", then the value is also the same as in beagle :)
Same for Cattle<=>Minipig comparison
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.
looks good to me
Fixes #1957 Add physiology data for the species Cattle (IV)
Fixes #293 Add physiology data for the species Cat