Skip to content
This repository has been archived by the owner on May 9, 2024. It is now read-only.

First pass at double-ended reconstruction for test beam MC and data #64

Merged
merged 16 commits into from
May 17, 2023
Merged

First pass at double-ended reconstruction for test beam MC and data #64

merged 16 commits into from
May 17, 2023

Conversation

cmantill
Copy link
Collaborator

  • Adding TOA conditions
  • Fixes bug on TOT reconstruction (was missing Matt's calibration to MIP equivalent)
  • Adds SingleEnded branches to the CalorimeterHit (we may get rid of those later)

@cmantill cmantill requested a review from mrsolt May 17, 2023 18:32
mrsolt
mrsolt approved these changes May 17, 2023
View reviewed changes
Copy link
Contributor

@mrsolt mrsolt left a comment

Choose a reason for hiding this comment

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

These changes look good to me.

@cmantill
Copy link
Collaborator Author

OK! I will merge and we can integrate fixes as they come along.

@cmantill cmantill merged commit 65f5b91 into trunk May 17, 2023
@cmantill cmantill deleted the iss40-toa branch May 17, 2023 19:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mrsolt mrsolt mrsolt approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cmantill @mrsolt