Skip to content
Snippets Groups Projects

Updating jFEX monitoring to add invalid code mapping

Merged Sergi Rodriguez Bosca requested to merge serodrig/athena:monitoring into master

This MR is meant to update the jFEX monitoring to include Invalid code mapping coming from LATOME. Also includes changes in the Phi coordinate to match the actually Tower size (Pi/32)

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
  • Looks ok but not sure what to make of the changes marked in a comment above so asking L2 to double check.

    Cheers,

    L1

    Edited by Kacper Wojciech Topolnicki
  • added 1 commit

    Compare with previous version

  • Sergi Rodriguez Bosca resolved all threads

    resolved all threads

  • added 1 commit

    Compare with previous version

  • There was a glitch on a build machine. Restarting the CI job

  • Jenkins please retry a build

  • This merge request affects 1 package:

    • Trigger/TrigT1/TrigT1CaloMonitoring

    This merge request affects 4 files:

    • Trigger/TrigT1/TrigT1CaloMonitoring/python/JfexInputMonitorAlgorithm.py
    • Trigger/TrigT1/TrigT1CaloMonitoring/python/JfexMonitorAlgorithm.py
    • Trigger/TrigT1/TrigT1CaloMonitoring/src/JfexInputMonitorAlgorithm.cxx
    • Trigger/TrigT1/TrigT1CaloMonitoring/src/JfexInputMonitorAlgorithm.h

    Adding @thompson ,@serodrig as watchers

  • :white_check_mark: CI Result SUCCESS (hash 53b571d5)

    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-CC7 62951]

  • Jenkins please retry a build

  • :pencil: Build area was cleaned as per request posted in the DB. The full software build will be performed

  • This merge request affects 1 package:

    • Trigger/TrigT1/TrigT1CaloMonitoring

    This merge request affects 4 files:

    • Trigger/TrigT1/TrigT1CaloMonitoring/python/JfexInputMonitorAlgorithm.py
    • Trigger/TrigT1/TrigT1CaloMonitoring/python/JfexMonitorAlgorithm.py
    • Trigger/TrigT1/TrigT1CaloMonitoring/src/JfexInputMonitorAlgorithm.cxx
    • Trigger/TrigT1/TrigT1CaloMonitoring/src/JfexInputMonitorAlgorithm.h

    Adding @thompson ,@serodrig as watchers

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