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

Fix when small_volfrac is used -- we need to adjust the data in cut c… #1451

Merged

Conversation

asalmgren
Copy link
Member

@asalmgren asalmgren commented Oct 13, 2020

Fix when small_volfrac is used -- we need to adjust the data in cut cell neighbors as well as regular neighbors

This addresses issue #1450

Summary

Additional background

Checklist

The proposed changes:

  • fix a bug or incorrect behavior in AMReX
  • add new capabilities to AMReX
  • changes answers in the test suite to more than roundoff level
  • are likely to significantly affect the results of downstream AMReX users
  • are described in the proposed changes to the AMReX documentation, if appropriate

@WeiqunZhang WeiqunZhang merged commit 5843b83 into AMReX-Codes:development Oct 14, 2020
WeiqunZhang added a commit to WeiqunZhang/amrex that referenced this pull request Oct 29, 2020
A GPU race condition was introduced in AMReX-Codes#1451 trying to fix faces and cells
next to covered cells that were converted from small cut cells.  It is fixed
by launching a separate kernel for each direction.
@WeiqunZhang WeiqunZhang mentioned this pull request Oct 29, 2020
5 tasks
asalmgren pushed a commit that referenced this pull request Oct 29, 2020
A GPU race condition was introduced in #1451 trying to fix faces and cells
next to covered cells that were converted from small cut cells.  It is fixed
by launching a separate kernel for each direction.
@asalmgren asalmgren deleted the fix_for_small_volfrac branch November 10, 2020 15:30
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