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

New UL cards for Run2 VV samples #3783

Merged
merged 6 commits into from
Dec 12, 2024
Merged

New UL cards for Run2 VV samples #3783

merged 6 commits into from
Dec 12, 2024

Conversation

toa-p
Copy link
Contributor

@toa-p toa-p commented Oct 17, 2024

Added new cards for 2016-2018 UL VV samples

define vl = ve vm vt
define vl~ = ve~ vm~ vt~
define ferm = u c d s b u~ c~ d~ s~ b~ e- mu- ta- e+ mu+ ta+ vl vl~
generate p p > h2, ( h2 > w+ w-, w+ > ferm ferm , w- > ferm ferm)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do you may be need another parenthesis after the w + w- for the separate w decays?

Copy link
Contributor Author

@toa-p toa-p Oct 22, 2024

Choose a reason for hiding this comment

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

Hi, this card is mostly a copy & paste from an older reference which does not use another parenthesis for separate W decays.

#*********************************************************************
# Run to generate the grid pack *
#*********************************************************************
.false. = gridpack !True = setting up the grid pack
Copy link
Collaborator

Choose a reason for hiding this comment

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

why?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Similar to the previous comment, this is a remnant from building the new card over the older reference. I am unsure of the motivation behind this line.

Copy link
Collaborator

@bbilin bbilin left a comment

Choose a reason for hiding this comment

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

Please see few comments. They apply also for the ZZ cards.

#*********************************************************************
'lhapdf' = pdlabel ! PDF set
$DEFAULT_PDF_SETS = lhaid
325300 = lhaid
Copy link
Collaborator

Choose a reason for hiding this comment

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

this should be removed. Above line should fetch the pdf and flavor etc for the given process.

Copy link
Contributor Author

@toa-p toa-p Oct 22, 2024

Choose a reason for hiding this comment

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

Thank you - I've taken note of this. I was (and still am) confused about where the LHAPDF ID should be specified when creating a sample request for these new UL VV samples. I am trying to follow this guide on MC production for UL campaigns, which recommended the above PDF set.

# meaningful ONLY if plotted taking matchscale *
# reweighting into account! *
#*********************************************************************
T = use_syst ! Enable systematics studies
Copy link
Collaborator

Choose a reason for hiding this comment

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

this is not needed, as it is overwritten.

@bbilin bbilin merged commit e0bab85 into cms-sw:master Dec 12, 2024
@bbilin
Copy link
Collaborator

bbilin commented Dec 12, 2024

We merge it, but please do a validation that the physics process is properly taken into account.

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.

2 participants