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

Ginkgo OMP problem #332

Closed
blegouix opened this issue Mar 8, 2024 · 6 comments · Fixed by #377
Closed

Ginkgo OMP problem #332

blegouix opened this issue Mar 8, 2024 · 6 comments · Fixed by #377
Assignees
Milestone

Comments

@blegouix
Copy link
Collaborator

blegouix commented Mar 8, 2024

Yann and me observed numerical issues in Gysela with OMP, even with OMP_NUM_THREADS=1.

#319 imputes the issue to Ginkgo with OMP executor.

I reproduce the issue by replacing:

std::size_t constexpr ncells = 100;

with:

std::size_t constexpr ncells = 32;

in tests/splines/non_periodic_spline_builder.cpp

(Note: there is no problem with ncells = 10 too, which is the setup for the batched tests.)

Then the splines_tests_DEGREE_X_3_BSPLINES_TYPE_UNIFORM_EVALUATOR_POLYNOMIAL_ tests will fail with error sometimes > 1e-10.

Doing the same in the #319 branch leads to all tests passing, with error usually around 1e-16.

@blegouix blegouix added this to the Splines milestone Mar 8, 2024
@blegouix blegouix self-assigned this Mar 8, 2024
@blegouix
Copy link
Collaborator Author

blegouix commented Mar 8, 2024

@tpadioleau
Copy link
Member

Can you do a test without the preconditioner ? It was not clear it was helpful and it could help determining whether the problem comes from the solver or the preconditioner.

@blegouix
Copy link
Collaborator Author

blegouix commented Mar 8, 2024

You are right it is the preconditionner. Using 1u (diagonal Jacobi) is enough to remove the problem.

@tpadioleau
Copy link
Member

see ginkgo-project/ginkgo#1563

@tpadioleau
Copy link
Member

Could we consider switching to the GMRES solver in the meantime ?

@blegouix
Copy link
Collaborator Author

blegouix commented Apr 2, 2024

#377

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 a pull request may close this issue.

2 participants