Skip to content
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

Open
KRiedmiller opened this issue Dec 21, 2023 · 4 comments
Open

aggregate_reactions hangs #355

KRiedmiller opened this issue Dec 21, 2023 · 4 comments
Labels
bug Something isn't working Priority

Comments

@KRiedmiller
Copy link
Collaborator

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.

@jmbuhr
Copy link
Collaborator

jmbuhr commented Dec 21, 2023

How big is very big?

@jmbuhr
Copy link
Collaborator

jmbuhr commented Dec 21, 2023

Looks like this part

kimmdy/src/kimmdy/recipe.py

Lines 373 to 383 in 09f364d

unique_recipes = []
unique_recipes_ixs = []
for i, recipe in enumerate(self.recipes):
if recipe.recipe_steps not in unique_recipes:
unique_recipes.append(recipe.recipe_steps)
unique_recipes_ixs.append([i])
else:
for j, ur in enumerate(unique_recipes):
if recipe.recipe_steps == ur:
unique_recipes_ixs[j].append(i)

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.

@KRiedmiller
Copy link
Collaborator Author

In fact, I had experimented with making them hashable, but decided that it looked too hacky :D Could be the better solution after all.
It fails for our big riken collagen system.

@jmbuhr
Copy link
Collaborator

jmbuhr commented Dec 21, 2023

Ah, I see! That is very big indeed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Priority
Projects
None yet
Development

No branches or pull requests

3 participants