-
Notifications
You must be signed in to change notification settings - Fork 189
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
Allocating multiple blocks to one mpi rank in LB #5026
base: python
Are you sure you want to change the base?
Conversation
… blocks pre mpi rank
testsuite/python/lb_couette_xy.py
Outdated
return v * u | ||
|
||
|
||
LB_PARAMS = {'agrid': 1., |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pls avoid 1s in unit tests, as wrong exponents don't get cauth.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I moved this function.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I changed the value of agrid from 1. to 0.6.
Thank you. Looks good in general. |
I rewrote the code related to the getters and setters for slices. Similar loops are pulled into a function which calls different lambdas for the individual cases. |
Description of changes:
blocks_per_mpi_rank
forLBFluidWalberla