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

feat: PFNano for Run2022 re-reco data and Run 3 Summer22(EE) MC #49

Merged
merged 3 commits into from
Feb 10, 2023

Conversation

demuller
Copy link

@demuller demuller commented Feb 9, 2023

Changes compared to 12_4_8 branch:

  • added/modified python config files for producing PFNano samples for re-reco Run2022 data (runs ABCD) and for Run 3 Summer22(EE) MC samples
  • adjusted example yml files
  • adjusted README.md
  • modified eventcontent and datatier for data from NANOAODSIM to NANOAOD
  • moved fakeNameforCrab modifier from pfnano_cff.py to cmsDriver commands

The XPOG recommendations for re-reco Run2022 data and Run 3 Summer2022(EE) MC can be found here

Notes:

  • a global tag recommendation for re-reco Run2022E exists and is different compared to ABCD, but could not be tested as the MINIAOD samples have not been produced yet, will be addressed in a follow-up PR
  • re-reco Run2022FG are currently WIP, will be addressed in a follow-up PR

@AnnikaStein AnnikaStein self-requested a review February 10, 2023 10:55
Copy link
Contributor

@AnnikaStein AnnikaStein left a comment

Choose a reason for hiding this comment

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

Thanks @demuller for the updated configs towards the new campaign!
Only left a few comments related to synchronizing across branches and cleaning up unused configs, but none of them are mandatory for merging.
Best,
A.

test/card_example_dataABCD.yml Show resolved Hide resolved
python/pfnano_cff.py Outdated Show resolved Hide resolved
@AnnikaStein
Copy link
Contributor

Thanks for the prompt updates, looks good now!

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