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

Threading does nothing for periodic FDSBP solvers #1026

Closed
jlchan opened this issue Dec 29, 2021 · 0 comments · Fixed by #1029
Closed

Threading does nothing for periodic FDSBP solvers #1026

jlchan opened this issue Dec 29, 2021 · 0 comments · Fixed by #1029
Labels
enhancement New feature or request performance We are greedy

Comments

@jlchan
Copy link
Contributor

jlchan commented Dec 29, 2021

Leaving this also as a reminder for me.

For periodic FDSBP operators implemented in the DGMulti framework, threads currently parallelize over elements. This is inefficient for periodic FDSBP operators, which only have a single element. We'll need to parallelize over nodes instead.

@jlchan jlchan added enhancement New feature or request performance We are greedy labels Dec 29, 2021
@ranocha ranocha linked a pull request Jan 11, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request performance We are greedy
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant