Skip to content

Light and overlay fixes — second try [develop] - #933

Merged
jas1005 merged 167 commits into
SBNSoftware:developfrom
PetrilloAtWork:feature/pmtsim_gainDevelopTakeTwo
Oct 28, 2026
Merged

jas1005 merged 167 commits into
SBNSoftware:developfrom
PetrilloAtWork:feature/pmtsim_gainDevelopTakeTwo

Conversation

@PetrilloAtWork

Copy link
Copy Markdown
Member

This is a reprise of PR #922, created rebasing commit e17d5cd to the current develop branch.

Unit tests have been run to verify the formal correctness of the FHiCL configurations and some more tests, however there has not been any physics revalidation.

P.S. I added a "fix" for the FHiCL unit test which failed in develop.

mvicenzi and others added 30 commits October 26, 2026 16:53
…plate

It used to end when the template SPR goes under 1e-4 ADC#.
Now it needs to stay within +/- 1w-4 ADC# for 20 nanoseconds.
This should make it possible to include undershootings.
All these parameters are currently hard-coded.
Good catches on names and strings.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@jas1005

jas1005 commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

trigger build LArSoft/lar*@LARSOFT_SUITE_v10_28_00 SBNSoftware/sbndaq-artdaq-core@v1_10_06

@FNALbuild

Copy link
Copy Markdown
Collaborator

✔️ CI build for LArSoft Succeeded on slf7 for c14:prof -- details available through the CI dashboard

@FNALbuild

Copy link
Copy Markdown
Collaborator

✔️ CI build for LArSoft Succeeded on slf7 for e26:prof -- details available through the CI dashboard

@FNALbuild

Copy link
Copy Markdown
Collaborator

❌ CI build for ICARUS Failed at phase build ICARUS on slf7 for c14:prof -- details available through the CI dashboard

🚨 For more details about the failed phase, check the build ICARUS phase logs

parent CI build details are available through the CI dashboard

@FNALbuild

Copy link
Copy Markdown
Collaborator

❌ CI build for ICARUS Failed at phase ci_tests ICARUS on slf7 for e26:prof - ignored warnings for build -- details available through the CI dashboard

🚨 For more details about the failed phase, check the ci_tests ICARUS phase logs

parent CI build details are available through the CI dashboard

@jas1005

jas1005 commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

I forgot that this requires a sequential CI build given the data product changes. I'll run that and update here.

@jas1005

jas1005 commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

Sequential build completes with only resource warnings. This is expected since release management has not merged the retuning PR. Proceeding with the merge!

@jas1005
jas1005 merged commit f92e629 into SBNSoftware:develop Oct 28, 2026
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants