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

GenAnalysis module added #34

Open
wants to merge 6 commits into
base: HZZ_Analysis
Choose a base branch
from
Open

GenAnalysis module added #34

wants to merge 6 commits into from

Conversation

YujiLee301
Copy link
Collaborator

No description provided.

@YujiLee301 YujiLee301 assigned ram1123 and unassigned ram1123 Jan 29, 2024
@YujiLee301
Copy link
Collaborator Author

GENZ_DaughtersId was omitted for now, as it looks difficult to get the access of a Daughter of Genpart directly from nanoAOD root file.

@ram1123 ram1123 linked an issue Jan 31, 2024 that may be closed by this pull request
@ram1123 ram1123 assigned YujiLee301 and unassigned ram1123 Jan 31, 2024
@YujiLee301
Copy link
Collaborator Author

Hi, @ram1123 @vukasinmilosevic Names of branch has been set in the thread. Could u please take a look if there should be something else to be added?

@YujiLee301
Copy link
Collaborator Author

Hi, @ram1123 @vukasinmilosevic. The sample list has been updated in this branch., including 2022 CDEFG era. Please take a look, thanks a lot!

self.out.branch("GENlep_Hindex", "I", lenVar = "GENHlepNum")
self.out.branch("lep_Hindex", "I", lenVar = "GENHlepNum")
self.out.branch("GENlep_id", "I", lenVar = "nGENLeptons")
self.out.branch("lep_genindex", "I", lenVar = "Lepointer")
Copy link
Owner

Choose a reason for hiding this comment

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

HI @YujiLee301 ,

Several branches are not in the output root file. You need to add them in the keep_and_drop.txt file, only then they will appear.

@ram1123 ram1123 linked an issue Feb 1, 2024 that may be closed by this pull request
@ram1123
Copy link
Owner

ram1123 commented Feb 1, 2024

Hi @YujiLee301 ,

I think you didn't added all the files that you changed. If I run over 2022 samples I get error. The reason for the error is that you are not adding info for 2022 in the post-proc.py

if "UL18" in first_file or "UL2018" in first_file:

Also, H4lCppModule() should be the first module added to the modules. Move this line before year if conditions

H4LCppModule = lambda: HZZAnalysisCppProducer(year,cfgFile, isMC, isFSR)
modulesToRun.extend([H4LCppModule()])

@YujiLee301
Copy link
Collaborator Author

Hi @ram1123 ,2022 options added here,but all the input is the same with 2018 as we do not have inputs like MuonRoccoR or SF .csv files of 2022.

Meanwhile, year is an important input of H4LCppModule, especially in the construction function. So I don't think we should move Line 84.

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

Successfully merging this pull request may close these issues.

2 participants