Skip to content

kfactor updates - #401

Open
MohitS704 wants to merge 5 commits into
CJLST:Run3from
MohitS704:Run3
Open

MohitS704 wants to merge 5 commits into
CJLST:Run3from
MohitS704:Run3

Conversation

@MohitS704

Copy link
Copy Markdown

No description provided.

@namapane namapane left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you @MohitS704 , it's great to have this ready!

I have added some comments of a few details that need to be fixed.
Also, as I mentioned ADD_LHE_PROB must be True to have LHE angles computed; given that this is mandatory when APPLY_K_NNLOQCD_ZZGG!=False, I would suggest to automatically set it here.

@bellan : FYI; please note that once merged this will affect your samples as well. I suppose you also need it for VBS - Please let us know if you have any comment!

Comment thread NanoAnalysis/python/nanoZZ4lAnalysis.py Outdated
Comment thread NanoAnalysis/python/weightFiller.py Outdated
Comment thread NanoAnalysis/python/weightFiller.py
Comment thread NanoAnalysis/python/weightFiller.py Outdated
@MohitS704

Copy link
Copy Markdown
Author

I have added some comments of a few details that need to be fixed. Also, as I mentioned ADD_LHE_PROB must be True to have LHE angles computed; given that this is mandatory when APPLY_K_NNLOQCD_ZZGG!=False, I would suggest to automatically set it here.

Note that this must be true when APPLY_K_NNLOQCD_NLOEW_ZZQQB, not APPLY_K_NNLOQCD_ZZGG. My understanding is the qqbar->ZZ samples are extremely large, so I have made it so that ADD_LHE_PROB is dependent on ADD_ALLEVENTS or APPLY_K_NNLOQCD_NLOEW_ZZQQB, so that the framework is not overburdened.

@namapane

namapane commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

I have added some comments of a few details that need to be fixed. Also, as I mentioned ADD_LHE_PROB must be True to have LHE angles computed; given that this is mandatory when APPLY_K_NNLOQCD_ZZGG!=False, I would suggest to automatically set it here.

Note that this must be true when APPLY_K_NNLOQCD_NLOEW_ZZQQB, not APPLY_K_NNLOQCD_ZZGG. My understanding is the qqbar->ZZ samples are extremely large, so I have made it so that ADD_LHE_PROB is dependent on ADD_ALLEVENTS or APPLY_K_NNLOQCD_NLOEW_ZZQQB, so that the framework is not overburdened.

Right - I copied the wrong flag!

Comment thread NanoAnalysis/python/weightFiller.py Outdated
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