Closed mjohnson541 closed 7 months ago
⚠️ One or more regression tests failed. Please download the failed results and run the tests locally or check the log to see why.
beep boop this comment was written by a bot :robot:
I tested this on the example in #2649 and it doesn't seem to be doing anything notable in the regression tests. However, this is a rather sensitive and opaque part of RMG.
In #2649 it seems like we are having issues with adding seed mechanism containing a matching initial species while the initial species are in the edge.
Because of how we process creating the initial core-edge and running estimation we add the initial species to the edge first later add the seed mechanisms and reaction libraries and lastly add the initial species to core and react them.
In #2649 we run into a problem because the function for adding seed mechanisms searches for "site" and finds it (it is in the edge), since it finds it, it seems to assume it doesn't need to do anything which becomes a problem when it tries to add the associated reaction directly to core since "site" isn't in the core.
I have modified add_seed_mechanism_to_core to track species that are in the edge and add them to core before adding the reactions. I have modified enlarge to be okay with reacting a species that has already been added to core.