Skip to content

A QC Task to try publishing different ROOT objects#984

Merged
Barthelemy merged 1 commit into
AliceO2Group:masterfrom
knopers8:every-object
Dec 8, 2021
Merged

A QC Task to try publishing different ROOT objects#984
Barthelemy merged 1 commit into
AliceO2Group:masterfrom
knopers8:every-object

Conversation

@knopers8
Copy link
Copy Markdown
Collaborator

@knopers8 knopers8 commented Dec 7, 2021

I used it to look for potential leaks in TCanvas, but this Task could be useful for similar investigations in the future, even if it does not cover every popular ROOT class yet.

I used it to look for potential leaks in TCanvas, but this Task could be useful for similar investigations in the future, even if it does not cover every popular ROOT class yet.
@knopers8 knopers8 requested a review from Barthelemy as a code owner December 7, 2021 14:10
Copy link
Copy Markdown
Collaborator

@Barthelemy Barthelemy left a comment

Choose a reason for hiding this comment

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

Very useful. One missing type perhaps, for a future iteration, is TTree.

@Barthelemy Barthelemy enabled auto-merge (squash) December 7, 2021 15:46
@Barthelemy
Copy link
Copy Markdown
Collaborator

@TimoWilken Is the mac builder ok ?

@TimoWilken
Copy link
Copy Markdown
Contributor

The Mac builder is up, it's just been testing a queue of PRs -- this PR got unlucky; the last check for new PRs was a minute before this one was submitted. It should be picked up soon.

@Barthelemy
Copy link
Copy Markdown
Collaborator

@TimoWilken Thanks a lot.

@Barthelemy Barthelemy merged commit b76121e into AliceO2Group:master Dec 8, 2021
Barthelemy pushed a commit to Barthelemy/QualityControl that referenced this pull request Jan 13, 2022
I used it to look for potential leaks in TCanvas, but this Task could be useful for similar investigations in the future, even if it does not cover every popular ROOT class yet.
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