Skip to content

Conversation

@nathanielerowe
Copy link
Contributor

The CI was picking up some differences between trigger objects caused by recent SBND specific changes. This should resolve one of the issues.

Depends on: SBNSoftware/sbnanaobj#158

@nathanielerowe nathanielerowe added the bugfix Addresses one or more bugs label Sep 17, 2025
@nathanielerowe
Copy link
Contributor Author

trigger build ci_ref=v10_09_00 LArSoft/lar*@LARSOFT_SUITE_v10_10_02 SBNSoftware/sbndaq_artdaq_core@v1_10_06 SBNSoftware/sbnd_data@v01_36_00 SBNSoftware/sbnanaobj#158

@kjplows kjplows self-requested a review September 17, 2025 20:04
Copy link
Contributor

@kjplows kjplows left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there's an implicit cast here - see https://github.com/SBNSoftware/sbnanaobj/blob/develop/sbnanaobj/StandardRecord/SRTrigger.h#L16

Is this change necessary for CI?

@nathanielerowe
Copy link
Contributor Author

trigger build ci_ref=v10_09_00 LArSoft/lar*@LARSOFT_SUITE_v10_10_02 SBNSoftware/sbndaq_artdaq_core@v1_10_06 SBNSoftware/sbnd_data@v01_36_00 SBNSoftware/sbnanaobj#158

@nathanielerowe
Copy link
Contributor Author

trigger build ci_ref=v10_09_00 LArSoft/lar*@LARSOFT_SUITE_v10_10_02 SBNSoftware/sbndaq_artdaq_core@v1_10_06 SBNSoftware/sbnd_data@v01_36_00 SBNSoftware/sbnanaobj#158

@nathanielerowe
Copy link
Contributor Author

trigger build ci_ref=v10_09_00 LArSoft/lar*@LARSOFT_SUITE_v10_10_02 SBNSoftware/sbndaq_artdaq_core@v1_10_06 SBNSoftware/sbnd_data@v01_36_00 SBNSoftware/sbnanaobj#158

@nathanielerowe
Copy link
Contributor Author

trigger build ci_ref=v10_09_00 LArSoft/lar*@LARSOFT_SUITE_v10_10_02 SBNSoftware/sbndaq_artdaq_core@v1_10_06 SBNSoftware/sbnd_data@v01_36_00 SBNSoftware/sbnanaobj#158

@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@FNALbuild
Copy link

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

@FNALbuild
Copy link

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

@FNALbuild
Copy link

❌ 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

❌ 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

@FNALbuild
Copy link

❌ 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

@FNALbuild
Copy link

⚠️ CI build for SBND Warning at phase ci_tests SBND on slf7 for e26:prof - ignored warnings for build -- 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

@linyan-w linyan-w moved this to Reviewer Approved in SBND 2025 Fall Production Sep 18, 2025
@linyan-w linyan-w moved this from Reviewer Approved to Waiting on Reviewer in SBND 2025 Fall Production Sep 18, 2025
@nathanielerowe nathanielerowe moved this from Waiting on Reviewer to Wait for next release (for data) in SBND 2025 Fall Production Sep 18, 2025
@linyan-w linyan-w moved this from Wait for validation II to Wait for next validation I patch (for MC) in SBND 2025 Fall Production Sep 19, 2025
@linyan-w linyan-w moved this from Wait for next validation I patch (for MC) to Wait for validation II in SBND 2025 Fall Production Sep 23, 2025
@kjplows kjplows moved this to To merge in SBN software development Sep 29, 2025
@kjplows kjplows merged commit 92596c4 into develop Sep 30, 2025
@github-project-automation github-project-automation bot moved this from To merge to Done in SBN software development Sep 30, 2025
@kjplows kjplows moved this from Done to Recently done in SBN software development Sep 30, 2025
@kjplows kjplows moved this from Recently done to Done in SBN software development Oct 1, 2025
@linyan-w linyan-w moved this from Wait for validation II to In tagged release in SBND 2025 Fall Production Oct 7, 2025
@nathanielerowe nathanielerowe deleted the bugfix/trigger_uint branch October 22, 2025 03:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Addresses one or more bugs

Projects

Status: Done
Status: In tagged release

Development

Successfully merging this pull request may close these issues.

4 participants