Open rwest opened 1 year ago
This is a result of the radical decay framework (https://github.com/ReactionMechanismGenerator/RMG-Py/blob/main/rmgpy/rmg/decay.py). The only current recipe is the Q.OOH. Broadly Q.OOH species either don't exist or are highly unstable so instead of forming them we should be automatically decaying them into Q=O + [OH]. However, it looks like somehow we sometimes are not doing that.
This was also noted in: https://github.com/ReactionMechanismGenerator/RMG-Py/discussions/2476.
This issue is being automatically marked as stale because it has not received any interaction in the last 90 days. Please leave a comment if this is still a relevant issue, otherwise it will automatically be closed in 30 days.
This is an apparent bug, adding to the project board for triage.
Bug Description
The regression test is reporting for liquid_oxidation-core.log
similarly for liquid_oidation-edge.log
The family
intra_H_migration
is meant to be intra-molecular Hydrogen migration, meaning a hydrogen atom migrates within a molecule. It should be a unimolecular reaction. The image isThe reaction above has two products:
Clearly it shouldn't come from intra_H_migration, like it claims.
Perhaps it comes from a pressure dependent network, one reaction of which is intra_H_migration?
So maybe the bug is just in the comparer tool used by the regression testing misinterpreting the comments on the reaction? Or perhaps the comments are being written incorrectly?
Anyway, there's a bug somewhere.
How To Reproduce
Run the liquid_oxidation regression test. Could also inspect the logs in the artifacts of any recent test run, eg. full results (31mb) or just the diff (15kb).
I suggest looking at the annotated chemkin file from the full logs to see what comment is being put on that reaction.
Expected Behavior
The results of the model diff wouldn't be confusingly attributing a bimolecular reaction to origin: intra_H_migration.