Skip to content

Iss451 calib banks#497

Merged
baltzell merged 17 commits intodevelopmentfrom
iss451-calibBanks
Aug 6, 2025
Merged

Iss451 calib banks#497
baltzell merged 17 commits intodevelopmentfrom
iss451-calibBanks

Conversation

@raffaelladevita
Copy link
Collaborator

No description provided.

@baltzell baltzell force-pushed the iss451-calibBanks branch from c26ba30 to 5153e14 Compare April 20, 2025 15:16
@raffaelladevita raffaelladevita marked this pull request as ready for review August 4, 2025 18:40
@raffaelladevita
Copy link
Collaborator Author

I modified the calib schema to remove banks that are not necessary anymore, but added back the ones used by timelines to the mon schema

@c-dilks
Copy link
Member

c-dilks commented Aug 4, 2025

Since this introduces a new module, could you add a new line to CODEOWNERS?

reconstruction/calib/ ...

baltzell
baltzell previously approved these changes Aug 5, 2025
@baltzell
Copy link
Collaborator

baltzell commented Aug 6, 2025

Seems it requires one of @c-dilks or @zieglerv to approve?

@baltzell baltzell merged commit 752de7e into development Aug 6, 2025
43 of 50 checks passed
@c-dilks c-dilks deleted the iss451-calibBanks branch August 6, 2025 20:36
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