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

LINEAR_SOLUTION_MODE=2 should be the default or the unit operation limit raised for auto #103

Closed
Immudzen opened this issue Sep 20, 2021 · 3 comments

Comments

@Immudzen
Copy link
Contributor

From what I can see with Industrial simulations that I have run and others have run (see forums https://forum.cadet-web.de/t/cadet-match-sma-simulation-not-converging/364/11) solving in parallel is almost always much slower. In the case of the forum the simulations ran about 10x faster by setting the mode to 2 and most people don't know about these settings and end up with very slow running simulations.

@sleweke-bayer
Copy link
Contributor

Do you recommend defaulting to linear mode completely, or do you recommend raising the barrier?
In case of the latter, what is your suggestion for the switching point?

@Immudzen
Copy link
Contributor Author

This is a good question. So far, I have not run into a single model where parallel solve was better when there were no cycles. I will ask Johannes if he has an example that runs better in parallel with no cycles. I suspect if you have multiple 2D GRM in a chain the linear solve would still be faster because you wouldn't have to repeat solve such expensive unit operations, but I have not tested that since I don't use 2D GRM.

@sleweke
Copy link
Member

sleweke commented Oct 27, 2021

I'm bumping the barrier up to 25. That is, sequential solution mode is selected for systems with up to (including) 25 unit operations. Please let me know if this is still insufficient.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants