Skip to content
Snippets Groups Projects

Fix for ATEAM-636

Merged Michele Renda requested to merge (removed):fix-ATEAM-636-3 into master
All threads resolved!

In this commit we made public the member of EventStreamInfo_p1.h to allow the extraction of the metadata values

Closes ATEAM-636

Edited by Michele Renda

Merge request reports

Merged by avatar (Mar 27, 2025 8:19am 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
  • Generally looks fine, I have a few minor style/clarity questions. Tom - L1 shifter

  • Michele Renda resolved all threads

    resolved all threads

  • Michele Renda added 1 commit

    added 1 commit

    Compare with previous version

  • This merge request affects 3 files:

    • Event/EventTPCnv/EventTPCnv/EventStreamInfo_p1.h
    • Tools/PyUtils/bin/meta-reader.py
    • Tools/PyUtils/python/MetaReader.py
  • :white_check_mark: CI Result SUCCESS (hash 0bbf1d7e)

    Athena AthSimulation AthGeneration AnalysisBase
    externals :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:
    make :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:
    optional tests :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
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 23085]

  • Hi @emoyse,
    Would it be possible to merge this one in over the weekend please, as it will allow me to make progress with !28950 (merged).
    Cheers,
    John

  • mentioned in commit eb0db7f6

  • Sorry @jchapman! I must have missed this request.

  • No problem @emoyse. I had plenty of other things to do. ;-) (Thanks for merging it @tsulaia and thanks again to @mrenda for the actual fix.)

  • John Derek Chapman mentioned in merge request !28950 (merged)

    mentioned in merge request !28950 (merged)

  • Please register or sign in to reply
    Loading