-
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
2113 property molecule psv #2117
Conversation
INSERT INTO tab_rate_generic_parameters(calculation_method,formula_rate,path_id,parameter_name,alias) VALUES('ExpressionParameters','PARAM_f_exp_bc_membrane',139,'Fraction expressed in blood cells','f_exp_bc_cell'); | ||
DELETE FROM tab_rate_generic_parameters WHERE calculation_method='ExpressionParameters' AND formula_rate='PARAM_f_exp_bc_membrane' AND path_id=240 AND parameter_name='Fraction expressed in blood cells'; | ||
INSERT INTO tab_rate_generic_parameters(calculation_method,formula_rate,path_id,parameter_name,alias) VALUES('ExpressionParameters','PARAM_f_exp_vasc_tissue_side',139,'Fraction expressed in endosomes','f_exp_vasend_endosomes'); | ||
INSERT INTO tab_rate_generic_parameters(calculation_method,formula_rate,path_id,parameter_name,alias) VALUES('ExpressionParameters','PARAM_f_exp_vasc_tissue_side',139,'Fraction expressed on plasma-side membrane of vascular endothelium','f_exp_vasend_plasma'); |
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.
.. instead of MOLECULE
propertyParameters.Each(x => | ||
{ | ||
var parameterPath = _entityPathResolver.ObjectPathFor(x); | ||
_psv[parameterPath].ShouldBeNull(); |
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.
do not add entries for property parameters (they should be excluded from psv)
@@ -89,13 +90,16 @@ private void setParameter(IParameter parameter, bool isSmallMolecule) | |||
if (parameter.HasAncestorNamed(CoreConstants.Compartment.ENDOSOME) && isSmallMolecule) | |||
return; | |||
|
|||
var parameterStartValue = getOrCreateStartValueFor(parameter); | |||
//We do not generate value for this parameter by default. Exit | |||
var parameterStartValue = getStartValueFor(parameter); |
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.
We were adding entries for formula that are the same everywhere. Not generated by default in Core and it should not be added here either
No description provided.