de-caplitalize Services and change the total tick count to 3427 for d… - #602
Conversation
|
trigger build RachelCoackley/larreco>larreco@v09_93_01rachel LArSoft/lar*@LARSOFT_SUITE_v09_93_01 SBNSoftware/sbncode@v09_93_01_p02 SBNSoftware/sbnanaobj@bug/coackley_ciCafv09_91_02 SBNSoftware/sbndaq-artdaq-core@v1_10_03 SBNSoftware/sbnobj@v09_19_05 |
bear-is-asleep
left a comment
bear-is-asleep
left a comment
There was a problem hiding this comment.
Previously approved
|
✔️ CI build for LArSoft Succeeded on slf7 for c14:prof -- details available through the CI dashboard |
|
❌ CI build for SBND Failed at phase build SBND on slf7 for c14:prof -- details available through the CI dashboard 🚨 For more details about the failed phase, check the build SBND phase logs parent CI build details are available through the CI dashboard |
|
✔️ CI build for LArSoft Succeeded on slf7 for e26:prof -- details available through the CI dashboard |
|
🚨 For more details about the warning phase, check the ci_tests SBND phase logs parent CI build details are available through the CI dashboard |
|
Approved - CI picks up some expected changes in CNN in CAFs |
4 participants
Description
this PR changes the default number of ticks to show for the raw digit display in the LArSoft event display from 3415 to 3427, as seen in recent data from run 17722, taken November 17. It also fixes a typo where the number of samples in evd_sbnd_data.fcl was in a block called "Services" when in fact it ought to be "services".
It does not affect the CAF data format, nor the standard workflow. Assigning Tingjun Yang as reviewer. No other PRs need to be merged.
This is a re-PR of PR #561, which had just a one-line change to the evd_sbnd_data.fcl config file, but which had conflicts merged over time that just made the git history very confusing. It was easier just to recreate the PR and close out PR #561
Checklist
Reviewers,AssigneesDevelopement