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

Add qqZZ sample for HIG-PODAS exercise #17

Open
wants to merge 8 commits into
base: podas
Choose a base branch
from

Conversation

bonanomi
Copy link
Contributor

No description provided.

@bonanomi bonanomi requested a review from pkausw September 27, 2023 13:48
Copy link
Contributor

@pkausw pkausw left a comment

Choose a reason for hiding this comment

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

Thanks @bonanomi , this looks great! Just some very minor comments here.

As discussed privately, I'll get on requesting a transfer of this and the signal sample in the near future. Thanks again for the work!

cmsdb/campaigns/run2_2018_nano_v11/ewk.py Show resolved Hide resolved
@@ -339,7 +339,7 @@
zz_llll = zz.add_process(
name="zz_llll",
id=8130,
xsecs={13: Number(0.1)}, # TODO
xsecs={13: Number(1.256)}, # TODO
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this the XS of XS(ZZ)*BR(Z->ll)**2? Might be good to write this out, or even have a look if e.g. the BR(Z->ll) is already part of the constants

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, this should be already xsec*BR. If we want to be 1:1 with the main HZZ analysis, we should consider adding the k-factors in the cf example to normalize this xsec to the NNLO one, but for the scope of the exercise this should be irrelevant.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hi @pkausw done in this commit

@riga
Copy link
Member

riga commented Mar 12, 2024

@bonanomi Is this PR still desired / alive, or did the POG exercise use your fork in the end?

@pkausw
Copy link
Contributor

pkausw commented Mar 12, 2024

Just to comment: The PODAS exercise used this specific branch in the end.

Nevertheless, since we want to repurpose the exercise for the columnflow demo, it might be interesting to merge these samples into cmsdb so we can always use the latest and greatest version in the demo. What do you think?

@bonanomi
Copy link
Contributor Author

Hi @riga , as mentioned by @pkausw we used this sample for the PODAS exercise. I am not sure if this qqZZ sample is going to be used by anyone else except for the HZZ4L example, but as @pkausw suggests, maybe this can be merged to make sure that we always use the latest greatest sample.

@riga
Copy link
Member

riga commented Mar 12, 2024

Sounds good. Then we should just merge after the above comment is resolved.

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