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

Remove NamedComponentParticleContainer (Use from AMReX) #5481

Merged
merged 53 commits into from
Feb 8, 2025

Conversation

atmyers
Copy link
Member

@atmyers atmyers commented Nov 22, 2024

@ax3l ax3l self-requested a review January 14, 2025 19:44
@ax3l ax3l added the component: core Core WarpX functionality label Jan 14, 2025
@ax3l ax3l self-assigned this Jan 18, 2025
@ax3l
Copy link
Member

ax3l commented Feb 4, 2025

I'll rebase and push a bit here while Andrew is OOO.

@ax3l ax3l force-pushed the removed_named_component branch from 74c9db2 to e95a324 Compare February 4, 2025 01:05
@ax3l ax3l marked this pull request as ready for review February 4, 2025 01:05
@ax3l ax3l force-pushed the removed_named_component branch from e95a324 to f77bba9 Compare February 4, 2025 01:05
@ax3l ax3l force-pushed the removed_named_component branch from f77bba9 to e174b85 Compare February 4, 2025 01:46
@ax3l ax3l changed the title [WIP] Remove NamedComponentParticleContainer from WarpX Remove NamedComponentParticleContainer from WarpX Feb 5, 2025
@ax3l ax3l changed the title Remove NamedComponentParticleContainer from WarpX Remove NamedComponentParticleContainer (Use from AMReX) Feb 5, 2025
@ax3l ax3l force-pushed the removed_named_component branch from 74414cd to ddfbedf Compare February 5, 2025 23:45
@ax3l
Copy link
Member

ax3l commented Feb 6, 2025

Oh, a few names in output files are still accidentially renamed:

Benchmark inner keys in plasma: dict_keys(['particle_momentum_x', 'particle_momentum_y', 'particle_momentum_z', 'particle_position_x', 'particle_weight'])
Test file inner keys in plasma: dict_keys(['particle_position_x', 'particle_ux', 'particle_uy', 'particle_uz', 'particle_w'])

@atmyers
Copy link
Member Author

atmyers commented Feb 8, 2025

Looks like the RZ tests are the only ones failing now.

@ax3l ax3l added the changes input scripts / defaults Changes the syntax or meaning of input scripts and/or defaults label Feb 8, 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.

Yayyy, nice!!

@ax3l
Copy link
Member

ax3l commented Feb 8, 2025

attn @roelof-groenewald @dpgrote @RemiLehe @clarkse this might slightly change Python scripts if you relied on adding runtime attributes (small change) or use the old naming queries (unlikely, but just want to raise).

@@ -141,7 +141,7 @@ Attribute name ``int``/``real`` Description Wher
Wheeler process physics is used.
==================== ================ ================================== ===== ==== ======================

WarpX allows extra runtime attributes to be added to particle containers (through ``NewRealComp("attrname")`` or ``NewIntComp("attrname")``).
WarpX allows extra runtime attributes to be added to particle containers (through ``AddRealComp("attrname")`` or ``AddIntComp("attrname")``).
Copy link
Member

Choose a reason for hiding this comment

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

Most likely break that happens in user code:
NewRealComp -> AddRealComp
NewIntComp -> AddIntComp

@ax3l ax3l enabled auto-merge (squash) February 8, 2025 06:10
@ax3l ax3l merged commit c0eacd9 into ECP-WarpX:development Feb 8, 2025
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes input scripts / defaults Changes the syntax or meaning of input scripts and/or defaults component: core Core WarpX functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants