Skip to content

better estimate of pileup - #34

Open
javierggt wants to merge 12 commits into
masterfrom
pileup
Open

better estimate of pileup#34
javierggt wants to merge 12 commits into
masterfrom
pileup

Conversation

@javierggt

Copy link
Copy Markdown
Contributor

Description

This is a small improvement in the pileup estimate. The pileup estimate in master uses the image generated from filtered events. That can cause an underestimate of the pileup. This is actually a tiny effect, which is relevant only for sources right at the edge of the selected field of view.

Pileup is still greatly underestimated in cases with large pileup (much more than what this change improves).

Interface impacts

Testing

Unit tests

  • No unit tests
  • Mac
  • Linux
  • Windows

Independent check of unit tests by [REVIEWER NAME]

  • [PLATFORM]:

Functional tests

No functional testing.

@javierggt
javierggt changed the base branch from master to improvements April 7, 2025 22:34
@javierggt
javierggt force-pushed the improvements branch 2 times, most recently from 03444e8 to 653c83b Compare August 5, 2025 16:09
Base automatically changed from improvements to master August 7, 2025 18:05
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.

1 participant