Add field binning to binning plugin #5265
Draft
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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 methodaddFieldBinner
Old
addBinner
method is renamed toaddParticleBinner
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
Binner
into a separate fileImportant for users, field binning has its own
DomainInfo
object. In addition to everything from before it also holds thelocalCellIndex
in the supercell and has a method togetCellIndex<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