Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

larpandoracontent v04_13_01 #72

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

AndyChappell
Copy link

This PR for larpandoracontent version v04_13_01 rolls back unintended changes to the vertex refinement algorithm introduced in v04_13_00 that affected SBND workflows.

@FNALbuild
Copy link

A new Pull Request was created by @AndyChappell for develop.

It involves the following packages:

larpandoracontent

@LArSoft/level-2-managers, @LArSoft/level-1-managers can you please review it and eventually sign? Thanks.

cms-bot commands are listed here

@FNALbuild
Copy link

The code-checks are being triggered in jenkins.

@FNALbuild
Copy link

+code-checks

@absolution1
Copy link

trigger build

@FNALbuild
Copy link

The tests are being triggered in jenkins.

@FNALbuild
Copy link

@FNALbuild
Copy link

@FNALbuild
Copy link

@FNALbuild
Copy link

@FNALbuild
Copy link

-SBND tests warning, with build warning,, with ignored warning for build, on slf7 for e26:prof
for details see
https://lar-ci-history.fnal.gov/LarCI/app/ns:SBND/view_builds/index?offset=0&builds=sbnd_ci/16234&builds=
for details of the parent CI build see
https://lar-ci-history.fnal.gov/LarCI/app/view_builds/index?offset=0&builds=lar_ci/25240&builds=

@FNALbuild
Copy link

@lgarren
Copy link
Member

lgarren commented Feb 24, 2025

approve

@FNALbuild
Copy link

This pull request is fully signed and it will be merged to develop and built in the next LArSoft release after it passes the integration tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Approved and pending inclusion in release
Development

Successfully merging this pull request may close these issues.

4 participants