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

produce lowPtGsfEle slimmed collections for the default miniAOD #25884

Closed
slava77 opened this issue Feb 7, 2019 · 13 comments
Closed

produce lowPtGsfEle slimmed collections for the default miniAOD #25884

slava77 opened this issue Feb 7, 2019 · 13 comments

Comments

@slava77
Copy link
Contributor

slava77 commented Feb 7, 2019

This is a follow up to #25753 .
During the review of this PR the new slimmed low-pt electron collections were added to the miniAOD only for bParking era/modifier due to a significant size increase for the default setup.
This issue should aim for a working low-pt electron setup in the default miniAOD to make use of the products created in RECO and saved in AOD.

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 7, 2019

A new Issue was created by @slava77 Slava Krutelyov.

@davidlange6, @Dr15Jones, @smuzaffar, @fabiocos, @kpedro88 can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@slava77
Copy link
Contributor Author

slava77 commented Feb 7, 2019

assign reconstruction,xpog

@Sam-Harper @lsoffi @bainbrid @mverzett
I consider this as a somewhat low priority/best-effort case and perhaps to be implemented with some real feedback from the bParking data experience.

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 7, 2019

New categories assigned: xpog,reconstruction

@slava77,@peruzzim,@perrotta,@fgolf you have been requested to review this Pull request/Issue and eventually sign? Thanks

@slava77
Copy link
Contributor Author

slava77 commented Apr 16, 2020

@jainshilpi @lsoffi @afiqaize @SohamBhattacharya
was there any progress on this topic?

@jainshilpi
Copy link
Contributor

The work on finalizing the regression and ID MVA is ongoing. Once that is done (~a months time), it can be put in miniAOD.

@slava77
Copy link
Contributor Author

slava77 commented May 27, 2020

@jainshilpi @lsoffi @afiqaize @SohamBhattacharya
please update if there was some recent progress on this topic.
Thank you.

@bainbrid
Copy link
Contributor

Hi @slava77, I'm in touch with @jainshilpi and @lsoffi to figure out what's needed. We'll get back to you ASAP.

@slava77
Copy link
Contributor Author

slava77 commented Jul 10, 2020

@jainshilpi @lsoffi @afiqaize @SohamBhattacharya
please update if there was some recent progress on this topic.
Thank you.

ping.
AFAIK, some work has been done recently; it would be nice to know (updated) expected time to delivery.
Thank you.

@jainshilpi
Copy link
Contributor

Talking to Chiara, in ~2 weeks, they expect it to be done.

@slava77
Copy link
Contributor Author

slava77 commented Aug 21, 2020

Talking to Chiara, in ~2 weeks, they expect it to be done.

please update on the status; we are getting closer to the deadlines

@jainshilpi
Copy link
Contributor

(1) Francesca will be presenting on Monday on the energy regression. The code is already ready for 11X. The training is currently performed using 2018 samples. Once the UL 2018 samples are available, the training will be updated. In teh meantime, to create a PR, we understand the training should be put in the tags. We will discuss with AlcaDB group on Monday about the creation of these temporary tags so that PR can be done.

(3) ID is already been taken care of by @bainbrid here: #31080
We are following up as to what would be the best way to put in the low pT electron ID in miniAOD. userfloats seems to be the best option to us. @guitargeek has already provided a recipe so it should work.

@bainbrid
Copy link
Contributor

For (3) above, I'll add the userFloat today or early next week. We need the retraining of the model, which will start next week too (shouldn't take too long).

@slava77
Copy link
Contributor Author

slava77 commented Feb 12, 2021

+1

provided with #31220 (11_2_0) for a default miniAOD and #32372 for UL re-miniAOD in 10_6_X since CMSSW_10_6_20

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

No branches or pull requests

4 participants