Skip to content
Snippets Groups Projects

Egamma no fwd electrons without track for ITk

Merge request reports

Pipeline #7233428 passed

Pipeline passed for f8c81d0a on llewitt:egammaNoFwdElectronsWithoutTrackForITk

Approval is optional

Merged by Frank WinklmeierFrank Winklmeier 1 year ago (Apr 22, 2024 11:12am UTC)

Merge details

  • Changes merged into main with 994dffe9 (commits were squashed).
  • Deleted 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
  • :white_check_mark: CI Result SUCCESS (hash d4113fc4)

    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-EL9 8102] (remote access info)

  • Lucy Lewitt resolved all threads

    resolved all threads

  • Lucy Lewitt added 1 commit

    added 1 commit

    Compare with previous version

  • Author Developer

    Jenkins please retry a build

  • This merge request affects 1 package:

    • Reconstruction/egamma/egammaAlgs

    This merge request affects 1 file:

    • Reconstruction/egamma/egammaAlgs/src/egammaForwardBuilder.cxx

    Adding @goetz as watcher

  • :white_check_mark: CI Result SUCCESS (hash f8c81d0a)

    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-EL9 8118] (remote access info)

  • Author Developer

    @mungo, @mdubau, @ludovica and @christos Are we happy with this as it is for now? Can make additional changes in a follow up MR if so.

  • It looks ok to me

  • I am happy with this, let's proceed and we will see what is the change on reco

  • Lucy Lewitt marked this merge request as ready

    marked this merge request as ready

  • Lucy Lewitt changed title from Egamma no fwd electrons without track for i tk to Egamma no fwd electrons without track for ITk

    changed title from Egamma no fwd electrons without track for i tk to Egamma no fwd electrons without track for ITk

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