WIP: gapfind: Alternative GapFind algorithm which is more robust #209
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Does not depend on v_max, instead sets a maximum flux when solving this problem to 1. This reduces the distance between v_max and epsilon making the problem more robust. This is able to solve larger models that fail with the previous implementation. This respects reaction directionality but ignores any reaction flux limits. This also lowers the maximum flux which scales the solution. This means that some reactions may not be able to produce epsilon flux.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is one attempt at finding an alternative implementation of GapFind that works correctly on larger models instead of marking almost all compounds blocked.
Does not depend on v_max, instead sets a maximum flux when solving this problem to 1. This reduces the distance between
v_maxandepsilonmaking the problem more robust. This isable to solve larger models that fail with the previous implementation. This respects reaction directionality but ignores any reaction flux limits. This also lowers the maximum flux which scales the solution. This means that generally there could be some reactions that may not be able to produce epsilon flux.