Skip to content

Conversation

jmsexton03
Copy link
Contributor

Summary

  • added dereference needed for using ParticleIDWrapper object
  • added assertions to a test for checking

Additional background

  • Originally seen as the following error while debugging Artemis CI:
error: invalid conversion from ‘long unsigned int*’ to ‘uint64_t’ {aka ‘long unsigned int’} [-fpermissive]
  436 |         this->m_idcpu = wrapper.m_idata;
      |                         ~~~~~~~~^~~~~~~
      |                                 |
      |                                 long unsigned int*

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
  • include documentation in the code and/or rst files, if appropriate

@jmsexton03 jmsexton03 requested a review from atmyers August 21, 2025 17:13
@atmyers atmyers merged commit 9d2b749 into AMReX-Codes:development Aug 21, 2025
89 of 90 checks passed
@atmyers
Copy link
Member

atmyers commented Aug 21, 2025

Thanks!

jessdagostini added a commit to jessdagostini/amrex that referenced this pull request Aug 27, 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