Skip to content
Snippets Groups Projects

no longer running updateCurrentxx if LockMapCurrents is true for AtlasFieldCacheCondAlg

Merged R D Schaffer requested to merge ATLAS-EGamma/athena:fiedCache_fieldOffFix into master

Small update to no longer run updateCurrentxx if LockMapCurrents is true since the current scale factors are forced to be 1, and added a bit cleared printout for current sfs. @fwinkl @christos Related to discussion in https://its.cern.ch/jira/browse/ATLASRECTS-5604

Edited by R D Schaffer

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
  • This merge request affects 1 package:

    • MagneticField/MagFieldServices
  • R D Schaffer added 1 commit

    added 1 commit

    • 401a585d - moving to allow for LockMapCurrents = True the field to turn on and off

    Compare with previous version

  • This merge request affects 1 package:

    • MagneticField/MagFieldServices
  • :negative_squared_cross_mark: CI Result FAILURE (hash 336cd863)

    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 :o: :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 18612]

  • :negative_squared_cross_mark: CI Result FAILURE (hash 401a585d)

    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 :o: :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 18625]

  • CI failures unrelated to the MR. @schaffer can you please answer the comment?

  • git seems slows to pick up the commit ... let me try something

    Edited by Christos Anastopoulos
  • This merge request affects 1 package:

    • MagneticField/MagFieldServices
  • R D Schaffer added 183 commits

    added 183 commits

    • 401a585d...cd010919 - 179 commits from branch atlas:master
    • 34eae6fd - MagFieldServices: Use run-based IOVs for conditions objects
    • ea084427 - updating map alg to handle job opt setting of currents for online running when...
    • 89e7ef41 - added additional tests for cases with no sol or tor at first event
    • 23936c6f - Merge remote-tracking branch 'upstream/master' into fiedCache_fieldOffFix

    Compare with previous version

  • R D Schaffer added 1 commit

    added 1 commit

    • 07e57ab4 - since DCS has time-stamp IOV, for IOV when DCS currents say that solenoid or...

    Compare with previous version

  • This merge request affects 1 package:

    • MagneticField/MagFieldServices
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading