Skip to content
Snippets Groups Projects

Update BPHY13

Merged Xin Chen requested to merge xchen/athena:mytopic into 21.2
All threads resolved!

Update BPHY13.py to enable passing "doBlinding" user flag via "--preExec" option to Reco_tr.py. The associated JIRA: https://its.cern.ch/jira/browse/ATLBPHYS-159

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
  • Hi @xchen, this looks generally ok to me, I just have a few questions. Tom - L1 shifter

    Edited by Thomas James Neep
  • Xin Chen added 1 commit

    added 1 commit

    • 9f5becb8 - Updated BPHY13.py according to merge shifter's comments.

    Compare with previous version

  • This merge request affects 1 file:

    • PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys/share/BPHY13.py

    Adding @wwalko ,@abarton as watchers

  • :white_check_mark: CI Result SUCCESS (hash 9f5becb8)

    AthDerivation
    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 on this CI monitor view
    :white_check_mark: AthDerivation: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-GCC62-CC7 181]

  • Thomas James Neep resolved all threads

    resolved all threads

  • Thomas James Neep resolved all threads

    resolved all threads

  • :pencil: Two CI jobs are triggered by 21.2 MR events: for AthDerivation with gcc62 compiler and with gcc8 for AnalysisBase and AthAnalysis projects.

    • Developers and shifters need to wait for both results, sometimes for additional hour or two.
  • :white_check_mark: CI Result SUCCESS (hash 9f5becb8)

    AnalysisBase AthAnalysis
    externals :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark:
    make :white_check_mark: :warning:
    required tests :white_check_mark: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view
    :white_check_mark: AnalysisBase: number of compilation errors 0, warnings 0
    :warning: AthAnalysis: number of compilation errors 0, warnings 6
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 23238]

  • @boeriu @calpigia: This is only affecting derivations.

  • mentioned in commit b349ffe0

  • Please register or sign in to reply
    Loading