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

General speedup revisited #471

Closed
connie opened this issue Oct 19, 2015 · 5 comments
Closed

General speedup revisited #471

connie opened this issue Oct 19, 2015 · 5 comments

Comments

@connie
Copy link
Member

connie commented Oct 19, 2015

See #78 for the older version of this issue.

I've noticed that on my computer, the minimal example running on 534cf3a takes 42 seconds vs the latest master at 21b83e3 takes 47 seconds, making it 5 seconds slower. I imagine this slowdown will affect larger models as well. @KEHANG presented at group meeting some changes which can help speed things up. I think there are also portions of code that could be cythonized or re-optimized. Please use this issue to brainstorm ideas or provide new profiling data.

@nickvandewiele
Copy link
Contributor

#488 should make simulations with many edge reactions faster.

@nickvandewiele
Copy link
Contributor

cpu_eg1

The figures shows the CPU times of make eg1 of the last 300 Travis CI builds triggered by ReactionMechanismGenerator/RMG-Py, i.e. pull requests into master and new branches.

@nickvandewiele
Copy link
Contributor

memory_used_eg1

Same for RAM used by make eg1

@rwest
Copy link
Member

rwest commented Nov 12, 2015

@walsh-p and @sherman-sc ran this job for about 7 hours before it crashed (an ODE solver problem). It got to 126 core species.
Here is the RMG.profile.pdf

Two remarkably large chunks:

  • chemkin:1653:markDuplicateReaction 23.16% (23.17%) 135941×
  • method 'update' of 'rmgpy.molecule.molecule.Molecule' objects> 27.61% (27.62%) 7599900×

Can we speed them up or call them less frequently?
Looks like we applied reaction recipes 2.5M times, so ran updateConnectivityValues 5M times (on average 2 new products per reaction?), but also ran update 7.5M times?

@mliu49
Copy link
Contributor

mliu49 commented May 17, 2018

Speed is still a problem, but this issue is well out-of-date.

@mliu49 mliu49 closed this as completed May 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants