Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added the script for the Fastjet DJR calculation in python #3487

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

jobach18
Copy link

No description provided.

@menglu21
Copy link
Collaborator

Hi @jobach18 we have several scripts to get the djr in the same directory, what's new in your script

@jobach18
Copy link
Author

Hi @menglu21,
All of them just plot DJR values they get from the generator code (or private code). Since the DJRs are not saved in nanoaod format, one needs to calculate them themselves. Which is what this script does: It calculates the DJR for a nanoaod output by CMSSW.
We have found this important in the evaluation of herwig produced samples where we look at the merging. It's a draft PR anyway for now, since the results are not yet validated.
Best
Joern

@sihyunjeon
Copy link
Collaborator

@menglu21, markus @mseidel42 and I asked for this so that we can perhaps utilize it for nanogen datatiers as well for different generators (since djr comes from the pythia hook)

@menglu21
Copy link
Collaborator

I see, do you already have some validation plots between using existing DJR value and using fastjet method. Another comment is that could you make the code more user friendly, e.g., I will not assume everyone has coffea and mplhep is only available in recent CMSSW version

@menglu21 menglu21 marked this pull request as ready for review July 25, 2023 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants