Skip to content
Snippets Groups Projects

MdtCondDBData - Use sets instead of vectors for faster look-up

Merged Johannes Junggeburth requested to merge jojungge/athena:CondDataUpdate into 23.0
All threads resolved!

Replace all vectors in the MuonCondDbData by std::set

Edited by Johannes Junggeburth

Merge request reports

Checking pipeline status.

Approval is optional

Merged by Stefan RichterStefan Richter 2 years ago (Mar 17, 2023 1:38pm UTC)

Merge details

  • Changes merged into 23.0 with c6592480 (commits were squashed).
  • Did not delete 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
  • Johannes Junggeburth resolved all threads

    resolved all threads

  • Johannes Junggeburth changed target branch from master to 23.0

    changed target branch from master to 23.0

  • removed master label

  • This merge request affects 3 packages:

    • MuonSpectrometer/MuonConditions/MuonCondGeneral/MuonCondData
    • MuonSpectrometer/MuonConditions/MuonCondGeneral/MuonCondTest
    • MuonSpectrometer/MuonDigitization/MDT_Digitization

    Affected files list will not be printed in this case

    Adding @jchapman ,@rosati ,@wleight ,@stavrop ,@jojungge ,@apsallid ,@pscholer as watchers

  • :white_check_mark: CI Result SUCCESS (hash a34c778c)

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    tests :white_check_mark:

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 68182]

  • added 26 commits

    Compare with previous version

  • This merge request affects 3 packages:

    • MuonSpectrometer/MuonConditions/MuonCondGeneral/MuonCondData
    • MuonSpectrometer/MuonConditions/MuonCondGeneral/MuonCondTest
    • MuonSpectrometer/MuonDigitization/MDT_Digitization

    Affected files list will not be printed in this case

    Adding @jchapman ,@rosati ,@wleight ,@stavrop ,@jojungge ,@apsallid ,@pscholer as watchers

  • :white_check_mark: CI Result SUCCESS (hash 68590848)

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    tests :white_check_mark:

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 68189]

  • Magdalena Slawinska resolved all threads

    resolved all threads

  • Stefan Richter mentioned in commit c6592480

    mentioned in commit c6592480

  • Frank Winklmeier mentioned in merge request !61662 (merged)

    mentioned in merge request !61662 (merged)

  • Please register or sign in to reply
    Loading