-
Notifications
You must be signed in to change notification settings - Fork 1
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
aggregate_reactions hangs #355
Comments
How big is very big? |
Looks like this part Lines 373 to 383 in 09f364d
could be more efficient. We are iterating over the unique recipes multiple times. If we make recipe steps hashable we can check for uniqueness easier. |
In fact, I had experimented with making them hashable, but decided that it looked too hacky :D Could be the better solution after all. |
Ah, I see! That is very big indeed. |
For homolysis with a very big system, aggregate_reactions in the recipe_collection freezes KIMMDY for unknown duration.
Workaround is to remove the aggregate_reactions call in runmanager._decide_recipe and to not plot the rates.
For homolysis (which uses rf kmc), this should be no issue, but for HAT and extrande it might. Needs more investigation.
The text was updated successfully, but these errors were encountered: