Skip to content

Conversation

@ftouchte
Copy link
Collaborator

  • load raw hit cuts from ccdb
  • update the key value of the atof ccdb

- load raw hit cuts from ccdb
- update the key value of the atof ccdb
@ftouchte ftouchte requested a review from baltzell April 16, 2025 21:32
@RDupre
Copy link
Collaborator

RDupre commented Apr 17, 2025

Why do we have all these changes on the ATOF stuff? Are you sure you are not carrying some modifications unrelated to the pull request, or reverting another commit you have not implemented yet?

@ftouchte
Copy link
Collaborator Author

The change on the ATOF is wanted. It is something I wanted to change a long time ago. Before, I defined the same key as for the AHDC without knowing the range of the (sector, layer, component, order) in the ATOF.

@baltzell baltzell merged commit 20d8d3f into JeffersonLab:development Apr 17, 2025
90 of 118 checks passed
@ftouchte ftouchte deleted the ccdb/ahdc-hit-cuts branch May 13, 2025 15:25
c-dilks pushed a commit that referenced this pull request Sep 15, 2025
- load raw hit cuts from ccdb
- update the key value of the atof ccdb
tongtongcao pushed a commit that referenced this pull request Sep 22, 2025
- load raw hit cuts from ccdb
- update the key value of the atof ccdb
tongtongcao pushed a commit that referenced this pull request Nov 18, 2025
- load raw hit cuts from ccdb
- update the key value of the atof ccdb
tongtongcao pushed a commit that referenced this pull request Nov 21, 2025
- load raw hit cuts from ccdb
- update the key value of the atof ccdb
tongtongcao pushed a commit that referenced this pull request Dec 5, 2025
- load raw hit cuts from ccdb
- update the key value of the atof ccdb
tongtongcao pushed a commit that referenced this pull request Dec 5, 2025
- load raw hit cuts from ccdb
- update the key value of the atof ccdb
@baltzell baltzell added the rg-l label Dec 23, 2025
tongtongcao pushed a commit that referenced this pull request Dec 31, 2025
- load raw hit cuts from ccdb
- update the key value of the atof ccdb
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.

3 participants