Skip to content
Snippets Groups Projects

Muon view algs to access RoI collections inside their views

Merged Benjamin Michael Wynne requested to merge bwynne/athena:MuonRoIsInViews into master

I don't think it should create any actual differences, but the muon algorithms running inside views were often accessing the RoI collection outside of that view. I have now changed that.

Potentially could change results where there are multiple RoIs in the outside-view collection, but only one within the view?

Algorithms within a view accidentally accessing an out-of-view container is a potential source of error, although is fine if intended. I've been searching for accidental cases.

Merge request reports

Pipeline #1293131 passed

Pipeline passed for 708803d4 on bwynne:MuonRoIsInViews

Approval is optional

Merged by Walter LamplWalter Lampl 5 years ago (Dec 13, 2019 4:59pm UTC)

Merge details

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

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