Skip to content

Conversation

@RemiLehe
Copy link
Member

@RemiLehe RemiLehe commented Jun 9, 2023

Now that #3777 is merged, it should be possible to do simulations with 2 levels of refinement.

@RemiLehe
Copy link
Member Author

RemiLehe commented Jun 9, 2023

The results of the test look correct, but the error (~5%) is slightly too high at this point.
Screenshot 2023-06-09 at 12 51 56 PM

@RemiLehe RemiLehe changed the title [WIP] Add test for MR with 2 levels Add test for MR with 2 levels Aug 11, 2023
@ax3l ax3l requested a review from atmyers August 14, 2023 20:10
@ax3l
Copy link
Member

ax3l commented Sep 25, 2023

attn @WeiqunZhang @atmyers

@ax3l
Copy link
Member

ax3l commented Sep 25, 2023

@RemiLehe oops, looks like this does not pass the test it adds. Can you please rebase and check? :)

@ax3l ax3l changed the title Add test for MR with 2 levels [WIP] Add test for MR with 2 levels Oct 2, 2023
@ax3l ax3l requested a review from WeiqunZhang December 1, 2025 17:22
@ax3l ax3l added the help wanted Extra attention is needed label Dec 1, 2025
@ax3l ax3l changed the title [WIP] Add test for MR with 2 levels Add test for MR with 2 levels Dec 2, 2025
Copy link
Member

@ax3l ax3l left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, the changes were small:
A tiny relaxation of the threshold for the analytical solution.

I added a checksum and the values on level 0 and 1 are also reasonably close to the 1 level MR case.

@ax3l ax3l removed the help wanted Extra attention is needed label Dec 2, 2025
@ax3l ax3l self-assigned this Dec 2, 2025
@ax3l ax3l enabled auto-merge (squash) December 2, 2025 01:31
@ax3l ax3l merged commit 2fd719c into BLAST-WarpX:development Dec 2, 2025
50 checks passed
@RemiLehe RemiLehe deleted the test_2_level_MR branch December 2, 2025 03:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants