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

Speed of calculations #479

Open
rs028 opened this issue Sep 26, 2022 · 3 comments
Open

Speed of calculations #479

rs028 opened this issue Sep 26, 2022 · 3 comments

Comments

@rs028
Copy link
Collaborator

rs028 commented Sep 26, 2022

The model is noticeably slower (compared to version pre-1.0). Need to do

  1. profiling to identify the bottlenecks and see how they can be eliminated.
  2. Is it worth considering parallelization?

Known to affect model performance:

  • time frequency and number of constraints.
  • size of the mechanism.

Unclear if affect model performance:

@rs028 rs028 added this to the version 1.4 milestone Oct 6, 2022
@rs028
Copy link
Collaborator Author

rs028 commented Nov 24, 2022

Results from a rough evaluation of the problem.
Model running on HPC system (30 GB memory), no constraints, 24 hours simulation.

n. species n. reactions runtime
51 137 15 sec
2575 7779 10 min
5833 17224 5.5 days

Increasing the memory to 120 GB or decreasing the output frequency do not affect significantly the outcome.

@ZhuJFDU
Copy link

ZhuJFDU commented Apr 20, 2023

entire mechanism = low efficiency

@rs028
Copy link
Collaborator Author

rs028 commented Oct 12, 2023

Changing the solver type in solver.parameters may help with this issue. The default choice is spgmr + banded preconditioner. The other choices (spgmr or dense) may result in better performance.

Note: Could this be related to #436? When lots of species have zero concentration the matrix is sparse and the use of preconditioner makes it inefficient?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Performance and Profiling
Development

No branches or pull requests

2 participants