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

Add field binning to binning plugin #5265

Draft
wants to merge 4 commits into
base: dev
Choose a base branch
from

Conversation

ikbuibui
Copy link
Contributor

@ikbuibui ikbuibui commented Feb 4, 2025

Add ability to bin fields

Quantity functors now get access to domain info as well, thus unifying the functor interface with the current one for axes as (auto const& domainInfo, auto const& worker, auto const& particle) Note: This breaks existing binning param files.

binningCreator has a new method addFieldBinner
Old addBinner method is renamed to addParticleBinner Note: This breaks existing binning param files.

Refactored the Binner into a parent binner class and specialized child particle and field binners.
Separated the helper kernels from Binnerinto a separate file

Important for users, field binning has its own DomainInfo object. In addition to everything from before it also holds the localCellIndex in the supercell and has a method to getCellIndex<ORIGIN> to get the current cell index being binned relative to an origin (global, total, local).

worker,domainInfo,particle instead

Move writeOpenPMDFunctor to set properties
setOpenPMDJsonCfg
setWriteOpenPMDFunctor

TODO : add this information to the documentation

@ikbuibui ikbuibui added the component: plugin in PIConGPU plugin label Feb 4, 2025
@ikbuibui ikbuibui added this to the 0.9.0 / next stable milestone Feb 4, 2025
@ikbuibui ikbuibui force-pushed the fieldBinning branch 8 times, most recently from 0a15c94 to e05375f Compare February 5, 2025 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: plugin in PIConGPU plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant