Skip to content

[ITS] add payload size plot to FEE task + minor improvements#949

Merged
Barthelemy merged 4 commits into
AliceO2Group:masterfrom
pfecchio:Fee-dev
Nov 22, 2021
Merged

[ITS] add payload size plot to FEE task + minor improvements#949
Barthelemy merged 4 commits into
AliceO2Group:masterfrom
pfecchio:Fee-dev

Conversation

@pfecchio
Copy link
Copy Markdown
Contributor

No description provided.

@pfecchio pfecchio changed the title Fee dev [ITS] add payload size plot to FEE task + minor improvements Nov 10, 2021
@iravasen
Copy link
Copy Markdown
Collaborator

@pfecchio please set nBins in the TH2 (y axis) in the json so that we can easily change it without going with another PR. Thank you!

@iravasen
Copy link
Copy Markdown
Collaborator

Thank you @pfecchio
@Barthelemy for us this PR can be merged.

@iravasen
Copy link
Copy Markdown
Collaborator

My comment arrived too early. Anyway I confirm it can be merged for us. Thank you

@Barthelemy
Copy link
Copy Markdown
Collaborator

@TimoWilken @ktf The status of the checks have been "waiting to be reported" since yesterday. Could you have a look ?

@knopers8
Copy link
Copy Markdown
Collaborator

@pfecchio Is a first time contributor, but I clicked the github button to allow the checks to run. Perhaps our CI does not respond to this?

@TimoWilken
Copy link
Copy Markdown
Contributor

Hi @Barthelemy, @knopers8, you're right, our CI doesn't respond to this. The checks will run if you approve the PR using the GitHub review functionality.

@iravasen
Copy link
Copy Markdown
Collaborator

@Barthelemy @knopers8 can this be merged? Unfortunately from the logs I cannot see if the issue with o2-cs8 is related to "arrow" as usual.

@Barthelemy Barthelemy merged commit 0f72c3b into AliceO2Group:master Nov 22, 2021
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.

5 participants