Skip to content

Conversation

@WeiqunZhang
Copy link
Member

This should wait for AMReX-Codes/amrex#4658.

@WeiqunZhang WeiqunZhang force-pushed the update_sum_boundary branch 2 times, most recently from c0c1eb6 to 145c430 Compare October 1, 2025 21:20
@WeiqunZhang
Copy link
Member Author

This is ready now.

@EZoni
Copy link
Member

EZoni commented Oct 6, 2025

@WeiqunZhang

A few CI checks did not run because the Ubuntu/CUDA one failed. Do you know how to fix it?

@EZoni EZoni self-assigned this Oct 6, 2025
@WeiqunZhang
Copy link
Member Author

I cannot reproduce it locally. It might be that I am using different versions of the compilers.

@WeiqunZhang
Copy link
Member Author

AMReX-Codes/amrex#4700 should fix it.

@WeiqunZhang
Copy link
Member Author

I think the fix does work. The HIP CI that is still using the old version is failing for the same reason as the CUDA one, which is passing now.

@EZoni
Copy link
Member

EZoni commented Oct 7, 2025

Thanks, @WeiqunZhang!

What about the CI check labeled apps / pyamrex that fails in AMReX-Codes/amrex#4700? Is that a new unrelated issue?

@WeiqunZhang
Copy link
Member Author

The CI in amrex uses the development branch of pyamrex, not this PR branch. So it is expected to fail.

@EZoni EZoni enabled auto-merge (squash) October 7, 2025 02:21
@EZoni EZoni merged commit 6f1b2dd into AMReX-Codes:development Oct 7, 2025
18 checks passed
@WeiqunZhang WeiqunZhang deleted the update_sum_boundary branch October 7, 2025 02:30
EZoni added a commit to BLAST-WarpX/warpx that referenced this pull request Oct 7, 2025
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