Closed rwest closed 14 years ago
Avoid expanding altered PDep networks with multiple reaction systems.
When running with multiple reaction systems, one reaction system may mark a species to be added to the core and another will mark a network to be expanded. Adding the species to the core could alter the network. When the network is then expanded, an error occurs because it has already been expanded (or at least altered).
We now check, before expanding a network, that it has not been altered already in this step.
Closed by ec0562d108416671732b04bf63c938dd44cc7709.
Just got the same error using latest version of code; reopening bug.
jing.rxn.PDepException: Tried to determine nonincluded isomer with maximum leak flux, but there are no nonincluded reactions, so no isomer can be identified. at jing.rxn.PDepNetwork.getMaxLeakIsomer(PDepNetwork.java:537) at jing.rxnSys.RateBasedPDepRME.enlargeReactionModel(RateBasedPDepRME.java:333) at jing.rxnSys.ReactionModelGenerator.enlargeReactionModel(ReactionModelGenerator.java:4029) at jing.rxnSys.ReactionModelGenerator.modelGeneration(ReactionModelGenerator.java:1415) at RMG.main(RMG.java:57) Tried to determine nonincluded isomer with maximum leak flux, but there are no nonincluded reactions, so no isomer can be identified. PDepNetwork #123: Isomers: CH3O(18) (included =true) CH2OH(19) (included =true) CH2O(20) + H(3) (included =true) CH2OJJ(139) + H(3) (included =true) O(4) + CH3(14) (included =true) Path reactions: CH3O(18) (included =true) --> CH2O(20) + H(3) (included =true) CH3O(18) (included =true) <=> CH2OJJ(139) + H(3) (included =true) CH3O(18) (included =true) --> CH2OH(19) (included =true) CH3O(18) (included =true) <=> O(4) + CH3(14) (included =true) CH2OH(19) (included =false) --> CH2O(20) + H(3) (included =true) CH2OH(19) (included =false) --> CH2OJJ(139) + H(3) (included =true) Net reactions: CH3O(18) (included =true) <=> CH2O(20) + H(3) (included =true) CH3O(18) (included =true) <=> CH2OJJ(139) + H(3) (included =true) CH3O(18) (included =true) <=> O(4) + CH3(14) (included =true) CH2O(20) + H(3) (included =true) <=> CH2OJJ(139) + H(3) (included =true) CH2O(20) + H(3) (included =true) <=> O(4) + CH3(14) (included =true) CH2OJJ(139) + H(3) (included =true) <=> O(4) + CH3(14) (included =true) CH3O(18) (included =true) --> CH2OH(19) (included =true) CH2O(20) + H(3) (included =true) --> CH2OH(19) (included =true) CH2OJJ(139) + H(3) (included =true) --> CH2OH(19) (included =true) Nonincluded reactions:
Adding reaction to Pdep network sets altered = true
Without this, networks are not reliably marked as altered when merged, and you can end up trying to expand networks that are already fully expanded.
Closed by ab92263524a7c2e5ba54bd9c4584eed411901316 (again!)
I just got this, and am not sure what to do about it: