Skip to content

FT0: add task for aging monitoring#2130

Merged
Barthelemy merged 1 commit into
AliceO2Group:masterfrom
andreasmolander:ft0AgingSquash
Feb 14, 2024
Merged

FT0: add task for aging monitoring#2130
Barthelemy merged 1 commit into
AliceO2Group:masterfrom
andreasmolander:ft0AgingSquash

Conversation

@andreasmolander
Copy link
Copy Markdown
Collaborator

This is a combination of 10 commits.

Development done by Andreas Molander, Sandor Lokos and Edmundo Garcia-Solis. Commit messages:

Aging monitoring with laser task implemented with an additional possibility to cut on amplitudes from config

CLang corrections done

Revision after MAC build fail

Mac build issue resolved

FT0: adapt to O2 coding guidelines and remove skeleton leftovers

[WIP] FT0: Refactor aging monitoring QC task

  • Some general refactoring, renaming etc.
  • Produce additional histograms (some only for debug, to be reviewed)
  • Delete obsolete stuff

[WIP] FT0: Refactor and add time histos in aging monitoring task

[WIP] FT0: refactor aging monitoring task

[WIP] FT0: minor fixes to aging QC + add README

[WIP] FT0: Clean up debug conditions in aging QC

This is a combination of 10 commits.

Development done by Andreas Molander, Sandor Lokos and Edmundo Garcia-Solis. Commit messages:

Aging monitoring with laser task implemented with an additional possibility to cut on amplitudes from config

CLang corrections done

Revision after MAC build fail

Mac build issue resolved

FT0: adapt to O2 coding guidelines and remove skeleton leftovers

[WIP] FT0: Refactor aging monitoring QC task

- Some general refactoring, renaming etc.
- Produce additional histograms (some only for debug, to be reviewed)
- Delete obsolete stuff

[WIP] FT0: Refactor and add time histos in aging monitoring task

[WIP] FT0: refactor aging monitoring task

[WIP] FT0: minor fixes to aging QC + add README

[WIP] FT0: Clean up debug conditions in aging QC
@andreasmolander
Copy link
Copy Markdown
Collaborator Author

Hello @jotwinow @afurs @knopers8 @Barthelemy,

Please have a look. It looks like codeowners of Modules/FIT are not pinged on changes in sub directories?

I think it looks like the failing tests are not related to this?

Copy link
Copy Markdown
Contributor

@jotwinow jotwinow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fine with me

@Barthelemy
Copy link
Copy Markdown
Collaborator

@andreasmolander It seems that people in the codeowner must have write access to the repo. I have added you. Be carefule please :-)

@Barthelemy
Copy link
Copy Markdown
Collaborator

spurious errors, merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants