Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a OneToMany relation from TrackerHit to RecIonizationCluster #211

Closed
wants to merge 1 commit into from

Conversation

BrieucF
Copy link
Contributor

@BrieucF BrieucF commented Jul 6, 2023

With the objects added in PR#179 for cluster counting algorithms, the only way to get the number of ionizations associated to a given TrackerHit is to do a matching by cellID which is quite inefficient. This PR proposes to add a link between TrackerHit and RecIonizationCluster to simplify the derivation of dN/dx.

BEGINRELEASENOTES

  • Add a OneToMany relation from TrackerHit to RecIonizationCluster to allow us deriving the number of clusters associated to a TrackerHit without having to do a matching by cellID

ENDRELEASENOTES

@BrieucF
Copy link
Contributor Author

BrieucF commented Jul 6, 2023

I would update the line numbers in the README.md once we agree on the way to go, before merging. By the way, isn't there a script somewhere that does it automatically?

@tmadlener
Copy link
Contributor

By the way, isn't there a script somewhere that does it automatically?

No, but there really should be ;)

@andresailer
Copy link
Collaborator

Closing (at least temporarily) while DriftChamber Tracker hit is under development by @BrieucF

@andresailer andresailer closed this Sep 5, 2023
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.

3 participants