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

Adding domain decomposition #129

Closed
wants to merge 50 commits into from
Closed

Adding domain decomposition #129

wants to merge 50 commits into from

Conversation

shac170
Copy link
Contributor

@shac170 shac170 commented Nov 9, 2023

Basic implementation of domain decomposition, no memory reduction yet.

@spasmann
Copy link
Collaborator

I wanted to update this PR because there had been some changes to main since it was opened. I had to merge with my fork's main though which is why the first three commits are the same from my iqmc pr/140.

After that, there were just a few changes that needed to be made to get DD running in numba mode again. I also added two regression tests. Currently configured, the tests only run if 4 MPI processes are present but we can change this to whatever we'd like, it just needs to be a specific number.

DD is reproducing results nicely on fixed source problems where there is only one processor per domain. If there is more than one per domain or the problem is k-eigenvalue I have not been able to reproduce results. @shac170 mentioned this is not expected behavior and is working to figure out the issues.

@shac170 shac170 closed this Jan 30, 2024
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.

2 participants