cms-analysis / flashgg

20 stars 158 forks source link

UL18 meta-conditions #1249

Closed rchatter closed 1 year ago

rchatter commented 3 years ago

[Do not merge] Added the UL18 S+S in the so called "old" format. Test run on GGH with the RR18 catalog [since the new UL18 catalog crashes for GGH].

Will keep updating this PR with more UL18 ingredients that are already available. Validation of this UL18 S+S in flashGG, derivation of shower shape corrections[@simonepigazzini ], vertex ID SFs[@youyingli ] can already begin on top of this PR.

rchatter commented 3 years ago

BTW just to add here the UL18 S+S file needs to be downloaded, so this is best merged on top of a fresh installation OR setup_flashgg.sh needs to be re-run at least line#54

maxgalli commented 3 years ago

Hi @rchatter, the shower shapes corrections for 2018 Legacy are ready and available in /eos/cms/store/group/phys_higgs/cmshgg/flashgg-data/Taggers/data/PhoIdInputsCorrections/2018_Legacy (regressors) and /eos/cms/store/group/phys_higgs/cmshgg/flashgg-data/Taggers/data/PhoIdInputsCorrections/corrections_summary_2018_Legacy.json (summary).

I'm thus going to add to this PR a commit which adds the updated path to MetaData/data/MetaConditions/Era2018_legacy_v1.json.

I have a question though: as suggested by @simonepigazzini, I performed the training on TnP datasets produced with also this change (https://github.com/maxgalli/flashgg/commit/0618a9664bc9768c27d8bc17ebf606f48037cc79). Should I include this commit as well?

rchatter commented 3 years ago

Hi @rchatter, the shower shapes corrections for 2018 Legacy are ready and available in /eos/cms/store/group/phys_higgs/cmshgg/flashgg-data/Taggers/data/PhoIdInputsCorrections/2018_Legacy (regressors) and /eos/cms/store/group/phys_higgs/cmshgg/flashgg-data/Taggers/data/PhoIdInputsCorrections/corrections_summary_2018_Legacy.json (summary).

I'm thus going to add to this PR a commit which adds the updated path to MetaData/data/MetaConditions/Era2018_legacy_v1.json.

I have a question though: as suggested by @simonepigazzini, I performed the training on TnP datasets produced with also this change (maxgalli@0618a96). Should I include this commit as well?

Hi Max, Sorry I missed this message yesterday. The global tags you have here (maxgalli@0618a96) are indeed the most up-to-date for the UL18 dataset. So please include this commit as well. Thanks for this fix!

Best, Rajdeep

rchatter commented 3 years ago

Hi You-Ying,

I wanted to add the UL18 Trigger efficiencies that @Soumyatifr derived. Since this PR is still open shall I do it on top of this one? Also have the UL18 Vertex SFs that you had derived been merged already?

Thanks, Rajdeep

youyingli commented 3 years ago

Hi @rchatter, yes, you can update trigger SF here before you fix the conflict. Once the SS correction is finished, you can also pull it to here. For me, I will also put the UL18 vtx SF here as well. Then, other materials like JEC/JER can be updated.

rchatter commented 3 years ago

Hi @rchatter, yes, you can update trigger SF here before you fix the conflict. Once the SS correction is finished, you can also pull it to here. For me, I will also put the UL18 vtx SF here as well. Then, other materials like JEC/JER can be updated.

Hi @youyingli , I submitted a fresh PR #1269 to add the Trigger ID SFs. Once that is merged @Prasant1993 can update the ID SFs and you can commit the the vtx SFs. Will then close out the remaining items. Thanks, Rajdeep

youyingli commented 1 year ago

1265 #1270 #1281 #1283 #1286 covered all UL18 meta-conditions updating. Close this PR, thanks!