Skip to content
Snippets Groups Projects

change 2j to 3j for photon chains and update the seed for a tau_xe chain (ATR-19317)

Merged Ligang Xia requested to merge ligang/athena:ATR-19317 into 21.3
All threads resolved!
  • g35_loose_L1EM30VHI_2j25 -> g35_loose_L1EM30VHI_3j25
  • g35_loose_L1EM30VHI_2j15_gsc25_boffperf_split -> g35_loose_L1EM30VHI_3j15_gsc25_boffperf_split
  • g35_loose_L1EM30VHI_2j15_gsc25_boffperf_split_FTK -> g35_loose_L1EM30VHI_3j15_gsc25_boffperf_split_FTK
  • change L1 seed of tau50_mediumRNN_tracktwoMVA_xe80_pufit_xe50_L1XE50 to L1_TAU20IM_2J20_XE50
  • add e24_lhmedium_nod0_mu8noL1 with L1_EM20VH
  • add j70_j50_0eta490_invm1000j70_dphi20_deta45_L1MJJ-500-NFF
Edited by Ligang Xia

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Author Developer

    Hi @tadej ,

    If you mean the commented line about "tau50_mediumRNN_tracktwoMVA_xe80_pufit_xe50_L1XE50", it is not needed. I did a test, and forgot to clean it. Sorry for inconvenience.

    Cheers, Ligang

    Edited by Ligang Xia
  • Ligang Xia resolved all discussions

    resolved all discussions

  • Ligang Xia added 1 commit

    added 1 commit

    • c4cff638 - remove one commented line in MC_pp_v8

    Compare with previous version

  • This merge request affects 1 package:

    • Trigger/TriggerCommon/TriggerMenu
  • :white_check_mark: CI Result SUCCESS

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    required tests :white_check_mark:
    optional tests :white_check_mark:

    Full details available at NICOS MR-25153-2019-08-12-18-10
    :warning: Athena: number of compilation errors 0, warnings 163
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST 40321]

  • Hi @lixia, a question from an L1. Can you elaborate on why you labeled this MR as review-pending-expert?

    Edited by Emily Petrova Takeva
  • Author Developer

    Hi @etakeva , Sorry, I removed the label. I thought the label was to remind the reviewers for checking, but it should be L1 revierwers to decide to add this label or not.

  • @lixia Okay, I will go ahead and approve now.

  • mentioned in commit ca00945c

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

    mentioned in merge request !26017 (merged)

  • mentioned in commit e5917b77

  • Please register or sign in to reply
    Loading