Skip to content

Conversation

@louisemillot
Copy link

Add detailed jet substructure analysis for data, MCD, and MCP

  • Implemented soft-drop jet reclustering with constituent handling
  • Filled matched histograms for geometry- and event-wise matched jets
  • Added event selection and centrality cuts for collisions in data and MC
  • Handled weighted MC events and event-wise constituent-subtracted jets
  • Updated histograms for leading/subleading prongs, zg, rg, and thetag observables
  • Added QA histograms for collisions and pTHat checks

@github-actions github-actions bot changed the title Jet substructure SoftDrop [PWGJE] Jet substructure SoftDrop Feb 11, 2026
WorkflowSpec defineDataProcessing(ConfigContext const& cfgc)
{
return WorkflowSpec{adaptAnalysisTask<JetSubstructureSoftDrop>(
cfgc, TaskName{"jet-substructure-softdrop"})}; // o2-linter: disable=name/o2-task(Required for correct DPL workflow)
Copy link
Collaborator

Choose a reason for hiding this comment

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

What do you mean by "Required for correct DPL workflow"?

Copy link
Author

Choose a reason for hiding this comment

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

I had initially tried without explicitly setting the TaskName, but in that case the workflow was not starting correctly in my DPL test setup. Forcing the TaskName made the device appear and run as expected. That’s why I added the override and the linter disable. I also followed the documentation, which suggested suppressing the linter in this case

Copy link
Collaborator

Choose a reason for hiding this comment

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

Your task name JetSubstructureSoftDrop should generate a device name jet-substructure-soft-drop. Did you observe anything else?

Comment on lines +41 to +42
#include "fastjet/ClusterSequenceArea.hh"
#include "fastjet/PseudoJet.hh"
Copy link
Collaborator

Choose a reason for hiding this comment

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

These are also external headers. Please include them using the <> format.

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

Development

Successfully merging this pull request may close these issues.

2 participants