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

EBData: New class containing Array4's for all EB data #4238

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

WeiqunZhang
Copy link
Member

This can make the code cleaner because we can get access to all available EB data in one object. This also reduces many GPU kernels' register pressure because the GPU device lambdas are much smaller without the need to capture multiple Array4's. For example, the 3D EBABecLap's gsrb kernel's number of registers reduces from 198 to 130.

Add a new function that returns a random point on EB for WarpX.

This can make the code cleaner because we can get access to all available EB
data in one object. This also reduces many GPU kernels' register pressure
because the GPU device lambdas are much smaller without the need to capture
multiple Array4's. For example, the 3D EBABecLap's gsrb kernel's number of
registers reduces from 198 to 130.

Add a new function that returns a random point on EB for WarpX.
@WeiqunZhang WeiqunZhang marked this pull request as ready for review December 3, 2024 22:28
@WeiqunZhang WeiqunZhang requested a review from atmyers December 3, 2024 22:28
@WeiqunZhang
Copy link
Member Author

Needed by ECP-WarpX/WarpX#5493

@WeiqunZhang WeiqunZhang merged commit 96db0a6 into AMReX-Codes:development Dec 6, 2024
75 checks passed
@WeiqunZhang WeiqunZhang deleted the eb_data branch December 6, 2024 18:45
WeiqunZhang added a commit to WeiqunZhang/amrex that referenced this pull request Dec 9, 2024
WeiqunZhang added a commit that referenced this pull request Dec 9, 2024
WeiqunZhang added a commit to WeiqunZhang/amrex that referenced this pull request Dec 21, 2024
This is a new function introduced in AMReX-Codes#4238.
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