Skip to content
Snippets Groups Projects

1st version of the top level reco driver script

Merged Tomasz Bold requested to merge tbold/athena:reco-top-jo into master

This is in fact a draft for people to comment. But we need something to allow us to asses the situation on this front and as well to tackle the technical issues. Two of them open for discussion:

  • should there be switches like doEgamma and the config to act accordingly?
  • how it is decided what objects are sent to the output?
Edited by Tomasz Bold

Merge request reports

Pipeline #2437058 passed

Pipeline passed for 25c3356e on tbold:reco-top-jo

Merged by Adam Edward BartonAdam Edward Barton 3 years ago (Mar 26, 2021 2:46pm 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
  • Tomasz Bold added 1 commit

    added 1 commit

    Compare with previous version

  • This merge request affects 1 package:

    • Reconstruction/RecJobTransforms

    This merge request affects 2 files:

    • Reconstruction/RecJobTransforms/CMakeLists.txt
    • Reconstruction/RecJobTransforms/python/RecoSteering.py

    Adding @goetz as watcher

  • Tomasz Bold marked this merge request as draft

    marked this merge request as draft

  • Tomasz Bold added 1 commit

    added 1 commit

    Compare with previous version

  • :white_check_mark: CI Result SUCCESS (hash 5fc8e907)

    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 :cloud: :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 29939]

  • :white_check_mark: CI Result SUCCESS (hash fb6760cf)

    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 29947]

  • removing L1 label while a draft. -Mike (L1)

  • Tomasz Bold added 1 commit

    added 1 commit

    Compare with previous version

  • Tomasz Bold resolved all threads

    resolved all threads

    • Author Developer
      Resolved by Tomasz Bold

      I have added an ID config. Taken from @jburr (who presumably has it from @iyeletsk & @zchubini - I do not know?). Anyways the job on BS with IT tracking doe not quite work. An error like this occurs:

      RIO_OnTrackErrorScalingCondAlg                         0     0   FATAL Unhandled attribute: PixPhi IBL
      RIO_OnTrackErrorScalingCondAlg                         0     0   ERROR Maximum number of errors ( 'ErrorMax':1) reached.

      with downstream consequences. My pick is to disable RAW test as well and let the experts to continue looking into respective issues.

      python -m RecJobTransforms.RecoSteering --ESD
      python -m RecJobTransforms.RecoSteering --RAW
  • Tomasz Bold added 246 commits

    added 246 commits

    Compare with previous version

  • Tomasz Bold added 1 commit

    added 1 commit

    Compare with previous version

  • Tomasz Bold added 1 commit

    added 1 commit

    Compare with previous version

  • Author Developer

    Teaser. The good news:

    ToolSvc.InDetTrackFitterAmbiOffline                               INFO finalize()
    ToolSvc.InDetTrackFitterAmbiOffline                               INFO 2765 attempted track fits
    ToolSvc.InDetTrackFitterAmbiOffline                               INFO 2731 successful track fits

    the not so good news:

    MuPatTrackBuilder.MuonTrackSteering.M...MCTBFitter                INFO finalize()
    MuPatTrackBuilder.MuonTrackSteering.M...MCTBFitter                INFO 0 attempted track fits
    MuPatTrackBuilder.MuonTrackSteering.M...MCTBFitter                INFO 0 successful track fits

    But this should let experts work out the details.

  • Tomasz Bold marked this merge request as ready

    marked this merge request as ready

  • This merge request affects 5 packages:

    • Calorimeter/CaloRec
    • Control/AthenaConfiguration
    • InnerDetector/InDetConfig
    • InnerDetector/InDetRecAlgs/InDetCaloClusterROISelector
    • Reconstruction/RecJobTransforms

    Affected files list will not be printed in this case

    Adding @goetz ,@amorley ,@pavol ,@sroe ,@ssnyder as watchers

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