Skip to content

[PWGJE] Minimum pT hat cut feature in jetSpectraEseTask - #18372

Open
joachimckh wants to merge 3 commits into
AliceO2Group:masterfrom
joachimckh:mc-jetspectraese-dev
Open

joachimckh wants to merge 3 commits into
AliceO2Group:masterfrom
joachimckh:mc-jetspectraese-dev

Conversation

@joachimckh

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the pwgje label Oct 20, 2026
@github-actions github-actions Bot changed the title Minimum pT hat cut feature in jetSpectraEseTask [PWGJE] Minimum pT hat cut feature in jetSpectraEseTask Oct 20, 2026
@github-actions

Copy link
Copy Markdown

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

@joachimckh
joachimckh marked this pull request as ready for review October 20, 2026 18:39
Comment thread PWGJE/Tasks/jetSpectraEseTask.cxx Outdated
registry.fill(HIST("mcd/hCentralitySel"), centrality);
float eventWeight = 1.0f;
if (cfgUseMCEventWeights) {
eventWeight = collision.has_mcCollision() ? collision.mcCollision_as<aod::JetMcCollisions>().weight() : 0.0f;

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.

by setting the weight of events that have no mcCollision to zero it makes pThat = inf which always passes your pThat cuts later I think

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.

Yes, I changed it to -1 and guard against these weights!

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.

2 participants