-
Notifications
You must be signed in to change notification settings - Fork 8
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
implement wrappy pFBA using the normal L2 wrappers #773
Conversation
@stelmo I've got 1 remaining test fail on this, which is actually on ECModel (the result is like 89% instd of 91% of some enzyme). Could you pls check what might have gone wrong? Otherwise this is IMO gtg |
Will look now, could be the solver misbehaving though |
# TODO here we would normally also overload the matrix function, but that | ||
# one will break once anyone touches variables of the models (which is | ||
# common). We should have a macro like @model_does_not_modify_variable_set | ||
# that adds the overloads. Or perhaps AbstractModelWrapperWithSameVariables? | ||
# | ||
# The same probably goes for other semantics; | ||
# AbstractModelWrapperThatOnlyTouchesSemantics(...) ? (Which has an | ||
# alternative in forcing people to overload all semantic functions in all | ||
# cases of adding semantics, which might actually be the right way.) | ||
|
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.
agreed
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## mo-pfba-pipe #773 +/- ##
================================================
+ Coverage 88.02% 88.63% +0.61%
================================================
Files 91 89 -2
Lines 2280 2262 -18
================================================
- Hits 2007 2005 -2
+ Misses 273 257 -16
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
No description provided.