Skip to content

Return actual statistics in sigma clipping aggregator#2841

Merged
maxnoe merged 5 commits into
mainfrom
fix-sigma-clip-stats
Sep 19, 2025
Merged

Return actual statistics in sigma clipping aggregator#2841
maxnoe merged 5 commits into
mainfrom
fix-sigma-clip-stats

Conversation

@mexanick
Copy link
Copy Markdown
Contributor

Fixes #2834

@mexanick mexanick added the fix label Sep 19, 2025
Comment thread src/ctapipe/monitoring/aggregator.py Outdated
Comment thread src/ctapipe/monitoring/aggregator.py Outdated
Comment thread src/ctapipe/monitoring/aggregator.py Outdated
mexanick and others added 3 commits September 19, 2025 15:59
Co-authored-by: Maximilian Linhoff <maximilian.linhoff@cta-observatory.org>
Co-authored-by: Maximilian Linhoff <maximilian.linhoff@cta-observatory.org>
Copy link
Copy Markdown
Member

@TjarkMiener TjarkMiener left a comment

Choose a reason for hiding this comment

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

Thanks!

@ctao-sonarqube
Copy link
Copy Markdown

@maxnoe maxnoe merged commit 181380f into main Sep 19, 2025
13 checks passed
@maxnoe maxnoe deleted the fix-sigma-clip-stats branch September 19, 2025 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SigmaClippingAggregator returns wrong sample size (n_events) value

3 participants