Skip to content
Snippets Groups Projects

NSWOfflineHelper class introduced to give the possibility to translate...

Merged Enrico Pasqualucci requested to merge (removed):ep/common-nsw-decoder into master

NSWOfflineHelper class introduced to give the possibility to translate detector ID and VMM number/channel to offline parameters and build offline IDs when no data word is available. The same class is now used by VMMChannel to insure coherency.

Merge request reports

Merged by avatar (Feb 19, 2025 6:03am 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
  • This merge request affects 1 package:

    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode

    This merge request affects 10 files:

    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/CMakeLists.txt_offline
    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/CMakeLists.txt_online
    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/MuonNSWCommonDecode/NSWDecodeHelper.h
    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/MuonNSWCommonDecode/NSWOfflineHelper.h
    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/MuonNSWCommonDecode/VMMChannel.h
    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/MuonNSWCommonDecode/sTGCMapper.h
    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/src/NSWOfflineHelper.cxx
    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/src/VMMChannel.cxx
    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/src/sTGCMapper.cxx
    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/src/test/test_nsw_online2offline.cxx

    Adding @rosati ,@wleight ,@stavrop ,@jojungge ,@egodden as watchers

  • :negative_squared_cross_mark: CI Result FAILURE (hash 2b62f9f9)

    Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon
    externals :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    required tests :o: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :white_check_mark: AthGeneration: number of compilation errors 0, warnings 0
    :white_check_mark: AnalysisBase: number of compilation errors 0, warnings 0
    :white_check_mark: AthAnalysis: 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 40016]

  • Jenkins please retry a build

  • This merge request affects 1 package:

    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode

    This merge request affects 10 files:

    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/CMakeLists.txt_offline
    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/CMakeLists.txt_online
    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/MuonNSWCommonDecode/NSWDecodeHelper.h
    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/MuonNSWCommonDecode/NSWOfflineHelper.h
    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/MuonNSWCommonDecode/VMMChannel.h
    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/MuonNSWCommonDecode/sTGCMapper.h
    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/src/NSWOfflineHelper.cxx
    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/src/VMMChannel.cxx
    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/src/sTGCMapper.cxx
    • MuonSpectrometer/MuonCnv/MuonNSWCommonDecode/src/test/test_nsw_online2offline.cxx

    Adding @rosati ,@wleight ,@stavrop ,@jojungge ,@egodden as watchers

  • :white_check_mark: CI Result SUCCESS (hash 2b62f9f9)

    Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon
    externals :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :white_check_mark: AthGeneration: number of compilation errors 0, warnings 0
    :white_check_mark: AnalysisBase: number of compilation errors 0, warnings 0
    :white_check_mark: AthAnalysis: 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 40066]

  • mentioned in commit ec6eff2a

Please register or sign in to reply
Loading