Skip to content

Conversation

@ax3l
Copy link
Member

@ax3l ax3l commented Feb 2, 2024

  • Add an explicit and GPU/CPU portable example to compute on the new, pure SoA particle layout.
  • Bindings: ParticleInitData for PureSoA
  • Remove legacy ImpactX and WarpX PC layout types.

Add an explicit and GPU/CPU portable example to compute
on the new, pure SoA particle layout.
@ax3l ax3l added component: documentation Improvements or additions to documentation enhancement New feature or request labels Feb 2, 2024
@ax3l ax3l requested a review from atmyers February 2, 2024 02:56
Comment on lines +391 to +392
.def("InitRandomPerBox", py::overload_cast<Long, ULong, const ParticleInitData&>(&ParticleContainerType::InitRandomPerBox))
.def("InitOnePerCell", &ParticleContainerType::InitOnePerCell)
Copy link
Member Author

Choose a reason for hiding this comment

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

@atmyers can you potentially add InitRandomPerBox and InitOnePerCell for the pure SoA layout in AMReX?

Copy link
Member Author

Choose a reason for hiding this comment

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

I tried it here, but would like to abandon at least the InitRandomPerBox one for a fresh try, since your InitRandom implementation was merged:
AMReX-Codes/amrex#3280

@ax3l ax3l force-pushed the doc-compute-pure-soa branch 3 times, most recently from 9e54910 to d4242a2 Compare February 2, 2024 03:46
@ax3l ax3l force-pushed the doc-compute-pure-soa branch from d4242a2 to 58be049 Compare February 2, 2024 03:50
@ax3l ax3l enabled auto-merge (squash) February 2, 2024 04:05
@ax3l ax3l disabled auto-merge February 2, 2024 04:05
@ax3l ax3l merged commit defb663 into AMReX-Codes:development Feb 2, 2024
@ax3l ax3l deleted the doc-compute-pure-soa branch February 2, 2024 04:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants