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

Refactoring ghost communicator #165

Open
fankiat opened this issue Jan 3, 2023 · 0 comments
Open

Refactoring ghost communicator #165

fankiat opened this issue Jan 3, 2023 · 0 comments
Assignees
Labels
prio:high High priority refactor Refactor existing codebase

Comments

@fankiat
Copy link
Owner

fankiat commented Jan 3, 2023

Currently, in #161 , the ghost communicator is quite convoluted and cluttered. While it works fine, it is error-prone and is probably hard to maintain. One idea to try for refactoring this is to abstract the indices used for the ghost communications, and let the ghost communication calls iterate over these indices and invoke the MPI send/recv calls accordingly.

@fankiat fankiat added refactor Refactor existing codebase prio:medium Medium priority labels Jan 3, 2023
@fankiat fankiat self-assigned this Jan 3, 2023
@fankiat fankiat added the prio:high High priority label Jan 5, 2023
@bhosale2 bhosale2 removed the prio:medium Medium priority label Jan 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
prio:high High priority refactor Refactor existing codebase
Projects
None yet
Development

No branches or pull requests

2 participants