Skip to content
Snippets Groups Projects

More fixes for missing muons collections with CA config

Merged Edward Moyse requested to merge emoyse/athena:r22-muon-ca-config-missing-collections into 22.0

This is a new configuration update to fix some missing collections when running with --CA. It adds back

  • MSDisplacedVertex
  • StauSegments
  • MuonTruthSegments etc

I also found a strange bug in MuonDetailedTrackTruthMaker - with CA, a default "" collection name is added, so then we get an off-by-one error with emplace_back

cc @rlangenb @jojungge

Edited by Edward Moyse

Merge request reports

Pipeline #4037543 passed

Pipeline passed for c0619d45 on emoyse:r22-muon-ca-config-missing-collections

Merged by Julien MaurerJulien Maurer 2 years ago (Jun 1, 2022 6:23pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • :negative_squared_cross_mark: CI Result FAILURE (hash 5fe166bd)

    Athena DetCommon
    externals :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark:
    required tests :o: :white_check_mark:
    optional tests :white_check_mark: :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
    :white_check_mark: DetCommon: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 53280]

  • Edward Moyse marked this merge request as draft

    marked this merge request as draft

  • Removing review label until this is ready to be reviewed.

    Kira (L1)

  • Edward Moyse added 2 commits

    added 2 commits

    • 00e19eeb - Add in Muon.makePRDs to disable some steps if re-running from ESD
    • 7e3fe26d - Add missing GEN_AOD2xAODCfg for standalone MC MuonConfig test

    Compare with previous version

  • Edward Moyse marked this merge request as ready

    marked this merge request as ready

  • Edward Moyse marked this merge request as draft

    marked this merge request as draft

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading