Skip to content
Snippets Groups Projects

Check calo bank size

Merged Check calo bank size
cmb_checkBnkSize into master
All threads resolved!
Merged Carla Marin Benito requested to merge cmb_checkBnkSize into master
All threads resolved!

Skip the bank if the data is empty

Needed to avoid crashes observed in the calo monitoring since 12.07, see eg. https://lblogbook.cern.ch/Shift/4287

FYI @jmarchan @dzuliani @gvouters

Edited by Carla Marin Benito

Merge request reports

Merge request pipeline #4245379 passed

Stage: test
    Stage: .post

      Merge request pipeline passed for d4a99752

      Approved by
      Sebastien Ponce

      Merged by Sebastien PonceSebastien Ponce 2 years ago (Jul 19, 2022 9:16am UTC)

      Merge details

      • Changes merged into master with 381c7fc6 (commits were squashed).
      • Deleted the source branch.

      Pipeline #4250542 passed

      Stage: test

        Pipeline passed for 381c7fc6 on master

        Activity

        Filter activity
        • Approvals
        • Assignees & reviewers
        • Comments (from bots)
        • Comments (from users)
        • Commits & branches
        • Edits
        • Labels
        • Lock status
        • Mentions
        • Merge request status
        • Tracking
      • Jean-Francois Marchand added 1 commit

        added 1 commit

        • 0fed4c4c - Modify pop to include isLittleEndian

        Compare with previous version

      • Jean-Francois Marchand added 1 commit

        added 1 commit

        Compare with previous version

      • Jean-Francois Marchand added 1 commit

        added 1 commit

        • 02f94be4 - Apply 1 suggestion(s) to 1 file(s)

        Compare with previous version

      • Jean-Francois Marchand added 1 commit

        added 1 commit

        • 6b4d0a33 - Apply 1 suggestion(s) to 1 file(s)

        Compare with previous version

      • Jean-Francois Marchand added 1 commit

        added 1 commit

        Compare with previous version

      • Jean-Francois Marchand added 1 commit

        added 1 commit

        • d4a99752 - Apply 1 suggestion(s) to 1 file(s)

        Compare with previous version

      • Carla Marin Benito resolved all threads

        resolved all threads

      • Carla Marin Benito marked this merge request as ready

        marked this merge request as ready

      • Carla Marin Benito changed the description

        changed the description

      • Carla Marin Benito
        Carla Marin Benito @cmarinbe ·
        Author Developer

        it's looking good, can we have a first round testing @msaur ? I can go tomorrow to the pit to test it with real data

      • Software for LHCb added ci-test-triggered label

        added ci-test-triggered label

      • L
        Software for LHCb @lhcbsoft ·
        Developer
      • Miroslav Saur assigned to @sponce

        assigned to @sponce

      • Sebastien Ponce approved this merge request

        approved this merge request

      • Sebastien Ponce resolved all threads

        resolved all threads

      • Sebastien Ponce merged

        merged

      • Sebastien Ponce mentioned in commit 381c7fc6

        mentioned in commit 381c7fc6

      • Miroslav Saur mentioned in issue Moore#456 (closed)

        mentioned in issue Moore#456 (closed)

        • Please register or sign in to reply