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

Fix/exasim meeting #111

Merged
merged 24 commits into from
Mar 1, 2024
Merged

Fix/exasim meeting #111

merged 24 commits into from
Mar 1, 2024

Conversation

greole
Copy link
Collaborator

@greole greole commented Feb 26, 2024

This PR symbol errors when compiling against the 64bit integer version of OF.

  • Additionally, it uses the presence of META-INFO to decide if the .com or .org version is used.
  • Adds more cmake presets.

@HendriceH HendriceH self-requested a review February 29, 2024 07:27
Copy link
Collaborator

@HendriceH HendriceH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything works now for me on HoreKa.
Actually the time passed after the second time step is even slightly faster when decomposing into 4 domains for 4 GPUs comparing to the full CPU node (103s vs 110s). However when decomposing into 8 domains for 4 GPUs its very slow again.

@greole
Copy link
Collaborator Author

greole commented Feb 29, 2024

Ok, thanks for testing!

However when decomposing into 8 domains for 4 GPUs its very slow again.
I think we have to live with that for now. Once we have the device repartitioning ready this should hopefully improve.

@greole greole merged commit 6a522fa into dev Mar 1, 2024
6 checks passed
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

Successfully merging this pull request may close these issues.

Solver crashes with symbol lookup error -> undefined symbol: _ZNK4Foam5UListINS_8UPstream11commsStructEEixEi
2 participants