Skip to content

Conversation

@stephenswat
Copy link
Member

This commit updates the truth matching code to have additional cuts on r, z, and the number of measurements on the track. It also adds some printouts to the finding efficiency writer to give the overall efficiency, fake rate, etc. without the need to use ROOT.

@stephenswat stephenswat added feature New feature or request examples Changes to the examples labels Jul 15, 2025
@stephenswat stephenswat force-pushed the feat/truth_finding_output branch from 1b0cd33 to e0b814b Compare July 22, 2025 09:00
@stephenswat
Copy link
Member Author

This is a prerequisite for #1082 and useful in general.

Copy link
Contributor

@niermann999 niermann999 left a comment

Choose a reason for hiding this comment

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

Looks good to me

Copy link
Member

@krasznaa krasznaa left a comment

Choose a reason for hiding this comment

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

Looks harmless enough.

This commit updates the truth matching code to have additional cuts on
r, z, and the number of measurements on the track. It also adds some
printouts to the finding efficiency writer to give the overall
efficiency, fake rate, etc. without the need to use ROOT.
@stephenswat stephenswat force-pushed the feat/truth_finding_output branch from e0b814b to f471b12 Compare July 23, 2025 15:03
@stephenswat stephenswat merged commit 82225b3 into acts-project:main Jul 23, 2025
21 of 27 checks passed
@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

examples Changes to the examples feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants