-
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
2132 better formulas #2157
2132 better formulas #2157
Conversation
@Yuri05 Just changed for one Rate. Can you check to make sure this is what you mean? |
@@ -3922,7 +3922,7 @@ INSERT INTO tab_calculation_method_rates VALUES('LinksCommon','ActiveEffluxSpeci | |||
INSERT INTO tab_calculation_method_rates VALUES('LinksCommon','ActiveEffluxSpecificWithTransporterInTarget_MM','TM * kcat * K_water_src * C_src / (Km + K_water_src * C_src)','Amount per time'); | |||
INSERT INTO tab_calculation_method_rates VALUES('LinksCommon','ActiveInfluxSpecificWithTransporterInSource_Hill','TM * kcat * (K_water_src * max(C_src;0))^alpha / (Km^alpha + (K_water_src * max(C_src;0))^alpha)','Amount per time'); | |||
INSERT INTO tab_calculation_method_rates VALUES('LinksCommon','ActiveInfluxSpecificWithTransporterInSource_MM','TM * kcat * K_water_src * C_src / (Km + K_water_src * C_src)','Amount per time'); | |||
INSERT INTO tab_calculation_method_rates VALUES('LinksCommon','BrainPgPToPlasma_Hill','Min(CP * f_int * V * kcat * (fu * max(C_pls;0))^alpha / ( Km^alpha + (fu * max(C_pls;0))^alpha) ; P_endothelial * SA * fu * C_pls) + CP * f_int * V * kcat * (K_water_int * max(C_int;0))^alpha / (Km^alpha + (K_water_int * max(C_int;0))^alpha)','Amount per time'); | |||
INSERT INTO tab_calculation_method_rates VALUES('LinksCommon','BrainPgPToPlasma_Hill','Min(TM * kcat * (fu * max(C_pls;0))^alpha / ( Km^alpha + (fu * max(C_pls;0))^alpha) ; P_endothelial * SA * fu * C_pls) + TM * kcat * (K_water_int * max(C_int;0))^alpha / (Km^alpha + (K_water_int * max(C_int;0))^alpha)','Amount per time'); |
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.
TM for Transporter Amount. This is what we use in another transports
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.
yep, looks good
@@ -85780,7 +85780,7 @@ INSERT INTO tab_rate_generic_molecules VALUES('LinksCommon','ActiveInfluxSpecifi | |||
INSERT INTO tab_rate_generic_molecules VALUES('LinksCommon','ActiveInfluxSpecificWithTransporterInSource_MM',366,'DRUG','TM',1); | |||
INSERT INTO tab_rate_generic_molecules VALUES('LinksCommon','BrainPgPToPlasma_Hill',341,'DRUG','C_int',0); | |||
INSERT INTO tab_rate_generic_molecules VALUES('LinksCommon','BrainPgPToPlasma_Hill',351,'DRUG','C_pls',0); | |||
INSERT INTO tab_rate_generic_molecules VALUES('LinksCommon','BrainPgPToPlasma_Hill',366,'DRUG','CP',0); | |||
INSERT INTO tab_rate_generic_molecules VALUES('LinksCommon','BrainPgPToPlasma_Hill',366,'DRUG','TM',1); |
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.
Use amount here
@@ -403,7 +403,7 @@ public static class Error | |||
public static string DistributionUnknown(string distribution) => $"Distribution '{distribution}' is unknown."; | |||
public const string NameIsRequired = "Name is required."; | |||
public const string MoleculeIsRequired = "Molecule is required."; | |||
public const string CategoryIsRequired = "Category is required."; | |||
public static readonly string CategoryIsRequired = $"{UI.ExpressionProfileCategory} is required."; |
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.
something else to get a better results
@Yuri05 All right. Should be all there. I am going to run the batch files to make sure there is no obvious corruption |
INSERT INTO tab_rate_generic_molecules VALUES('Observer_PKSim_2Poren','OBS_WholeOrgan_liv',4713,'DRUG','C_pls_prp',1); | ||
INSERT INTO tab_rate_generic_molecules VALUES('Observer_PKSim_2Poren','OBS_WholeOrgan_liv',4718,'DRUG','C_pls_prc',1); | ||
INSERT INTO tab_rate_generic_molecules VALUES('Observer_PKSim_2Poren','OBS_WholeOrgan_liv',4723,'DRUG','C_end_prp',1); | ||
INSERT INTO tab_rate_generic_molecules VALUES('Observer_PKSim_2Poren','OBS_WholeOrgan_liv',4728,'DRUG','C_end_prc',1); |
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.
Those entries are not required anymore and should be deleted
INSERT INTO tab_rate_generic_molecules VALUES('Observer_PKSim_2Poren','OBS_WholeOrgan_liv',4708,'DRUG','C_cell_prc',1);
INSERT INTO tab_rate_generic_molecules VALUES('Observer_PKSim_2Poren','OBS_WholeOrgan_liv',4713,'DRUG','C_pls_prp',1);
INSERT INTO tab_rate_generic_molecules VALUES('Observer_PKSim_2Poren','OBS_WholeOrgan_liv',4718,'DRUG','C_pls_prc',1);
INSERT INTO tab_rate_generic_molecules VALUES('Observer_PKSim_2Poren','OBS_WholeOrgan_liv',4723,'DRUG','C_end_prp',1);
INSERT INTO tab_rate_generic_molecules VALUES('Observer_PKSim_2Poren','OBS_WholeOrgan_liv',4728,'DRUG','C_end_prc',1);
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.
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.
the diff was off
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.
lol, ok
@msevestre ok, apart from https://github.com/Open-Systems-Pharmacology/PK-Sim/pull/2157/files#r843794150 looks good for me. When this is fixed we can merge and then I would run the batch comparison before we fix the pgp issue |
ok merging. |
No description provided.