Skip to content

[PWGDQ] add tag and probe method for dalitz selection#15948

Open
glegras wants to merge 5 commits intoAliceO2Group:masterfrom
glegras:dalitz
Open

[PWGDQ] add tag and probe method for dalitz selection#15948
glegras wants to merge 5 commits intoAliceO2Group:masterfrom
glegras:dalitz

Conversation

@glegras
Copy link
Copy Markdown
Contributor

@glegras glegras commented Apr 23, 2026

No description provided.

@github-actions github-actions Bot added the pwgdq label Apr 23, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 23, 2026

O2 linter results: ❌ 68 errors, ⚠️ 227 warnings, 🔕 0 disabled

@glegras glegras marked this pull request as ready for review April 23, 2026 16:53
@alibuild
Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 8926f02 at 2026-04-26 02:46:

## sw/BUILD/O2Physics-latest/log
c++: fatal error: Killed signal terminated program cc1plus
ninja: build stopped: subcommand failed.

Full log here.

Comment thread PWGDQ/Core/CutsLibrary.cxx Outdated
Comment on lines +5098 to +5101
TF1* fDCAxyresLow = new TF1("fDCAxyresLow", "[0] + [1] * pow(x, -[2])", 0.1, 1000.);
TF1* fDCAzresLow = new TF1("fDCAzresLow", "[0] + [1] * pow(x, -[2])", 0.1, 1000.);
TF1* fDCAxyresUp = new TF1("fDCAxyresUp", "[0] + [1] * pow(x, -[2])", 0.1, 1000.);
TF1* fDCAzresUp = new TF1("fDCAzresUp", "[0] + [1] * pow(x, -[2])", 0.1, 1000.);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

How do these objects get deleted?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Since they are used in the AnalysisCut, they cannot be deleted until the end. But now I modified so that these TF1 are defined only when necessary

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

You didn't answer my question.

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

Labels

Development

Successfully merging this pull request may close these issues.

3 participants