Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ITS - Fixes and improvement on FEE Task checker #2016

Merged
merged 2 commits into from
Oct 16, 2023
Merged

ITS - Fixes and improvement on FEE Task checker #2016

merged 2 commits into from
Oct 16, 2023

Conversation

nicolovalle
Copy link
Contributor

@iravasen @IsakovAD , this is on the checks for the TriggerVsFeeID plot

  • Fix: Before the checker took into account only the number of FEEs excluded from the checks, not their IDs
  • Feature: New check allowing to spot if a FEEId stays behind with the number of shipped TFs (e.g. module going off along the run)

@nicolovalle nicolovalle changed the title ITS - Fixes and improvememt on FEE Task checker ITS - Fixes and improvement on FEE Task checker Oct 15, 2023
@IsakovAD IsakovAD merged commit 808c9b6 into AliceO2Group:master Oct 16, 2023
2 checks passed
@nicolovalle nicolovalle deleted the nv-its-feechcker branch November 6, 2023 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@IsakovAD IsakovAD IsakovAD approved these changes

@iravasen iravasen Awaiting requested review from iravasen iravasen is a code owner

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nicolovalle @IsakovAD