Skip to content

Update PMT database tags [develop]#858

Closed
mvicenzi wants to merge 94 commits intodevelopfrom
feature/mvicenzi_run4_tcorr
Closed

Update PMT database tags [develop]#858
mvicenzi wants to merge 94 commits intodevelopfrom
feature/mvicenzi_run4_tcorr

Conversation

@mvicenzi
Copy link
Copy Markdown
Member

This PR updates the tags for the PMT calibration databases to PMT_CalibrationTags_Oct2025.
Changes:

  • New cosmics corrections table for Run-4.

It requires an updated pmt_cosmics_timing_data.db in icarus_data with the v2r3 tag.

gputnam and others added 30 commits May 16, 2025 11:11
keep PMT waveforms in stage and add fcl to drop them later
…T-metadata

Enable Metadata Injection for Production
@mvicenzi mvicenzi self-assigned this Oct 27, 2025
@mvicenzi
Copy link
Copy Markdown
Member Author

For reference: these new tags are now available in icarus_data v10_06_01
Thanks @jedori0228 @SFBayLaser !

@mvicenzi mvicenzi changed the title Update PMT database tags Update PMT database tags [develop] Nov 6, 2025
@jas1005
Copy link
Copy Markdown
Contributor

jas1005 commented Apr 3, 2026

@jedori0228 and @mvicenzi

I'm only tagging y'all since Tracy is a reviewer for many other open PRs. Do either of you know of another reviewer I could request in addition to Jaesung? This seems like a lot to review for one person.

It looks like this PR will need special attention since there are merge conflicts given recent upgrades to LArSoft. Note that even further updates may be needed after all of the work from release/SBN2025A is migrated over to develop.

@mvicenzi
Copy link
Copy Markdown
Member Author

mvicenzi commented Apr 3, 2026

Hi @jas1005 : this PR is a bit unfortunate, because this branch is already contained in release/SBN2025A (see #864 ).
So I'd say that if release/SBN2025A is migrated over to develop, there will be no need to merge this in.

Perhaps it would be just easier to close this one to avoid confusion.

@jas1005
Copy link
Copy Markdown
Contributor

jas1005 commented Apr 3, 2026

Sounds good! I'm also the person migrating the work on release/SBN2025A over to develop. I'll make sure these changes make it in. I'll close this PR then.

@jas1005 jas1005 closed this Apr 3, 2026
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.

7 participants