Skip to content
Snippets Groups Projects

Make VP A and C side more explicit with an enum struct

Merged Titus Mombaecher requested to merge tmombach_fixACside into 2024-patches
All threads resolved!

Following LHCb!4523 (merged) where a confusion between A and C side appeared we discussed it may be best to make the A and C side explicit with a struct instead of an int.

This MR goes together with Rec!3877 (merged) and LHCb!4523 (merged)


Validated by

  • Core Software
  • RTA
  • Simulation
Edited by Sebastien Ponce

Merge request reports

Merge request pipeline #7278321 passed

Merge request pipeline passed for ee0debcf

Merged by Christopher Rob JonesChristopher Rob Jones 11 months ago (Apr 25, 2024 12:31pm UTC)

Merge details

  • Changes merged into 2024-patches with 5826d9fa (commits were squashed).
  • Deleted the source branch.

Pipeline #7285971 passed

Pipeline passed for 5826d9fa on 2024-patches

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