Skip to content

Jpkneller add collate and aggregate functions for rate calculator tables - #445

Open
jpkneller wants to merge 21 commits into
mainfrom
jpkneller-add_collate_for_RateCalculator
Open

Jpkneller add collate and aggregate functions for rate calculator tables#445
jpkneller wants to merge 21 commits into
mainfrom
jpkneller-add_collate_for_RateCalculator

Conversation

@jpkneller

Copy link
Copy Markdown
Contributor

Add collate function to process tables returned by RateCalculator.run

jpkneller added 3 commits July 7, 2026 11:40
Add a collate function to collate the rates returned by RateCalculator.run.
@jpkneller
jpkneller marked this pull request as ready for review July 7, 2026 17:11

@sybenzvi sybenzvi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I have no issues with the new collate calculation, but note that operator+ was newly defined for the flux containers. No issue with that either, but @Sheshuk should please weigh in.

@jpkneller

Copy link
Copy Markdown
Contributor Author

@sybenzvi, @Sheshuk, I added another function called aggregate which also uses the + operator. The purpose of aggregate is to sum over all channels for a given detector, whereas collate only sums over channels where the name matches a specified pattern. e.g. nc_*

@jpkneller jpkneller changed the title Jpkneller add collate for rate calculator Jpkneller add collate and aggregate functions for rate calculator tables Jul 9, 2026
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