Skip to content
Snippets Groups Projects

Make comparison in warning printout more robust

Merged Marco Cattaneo requested to merge fixCaloDoDWarning-master into master
All threads resolved!

In the check for duplicate algorithms in the configuration of DataOnDemand, the Calo RawBank decoding algorithms are returned as objects, not strings. This was causing the check to always fail, which resulted in always printing out the entire configuration of the algorithm.

Cherry-picked from !3015 (merged) on run2-patches. I don't know if the DoD is still relevant in master, but as long as this file is there it probably makes sense to fix it...

Merge request reports

Pipeline #2535853 passed

Pipeline passed for f7bf58fc on fixCaloDoDWarning-master

Approval is optional

Merged by Rosen MatevRosen Matev 3 years ago (Apr 28, 2021 9:30pm UTC)

Merge details

  • Changes merged into with 3676b57a.
  • Deleted the source branch.

Pipeline #2547599 passed

Pipeline passed for 3676b57a 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
Please register or sign in to reply
Loading