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

features for UL re-miniAOD in 10_6_X #27889

Closed
41 tasks done
slava77 opened this issue Aug 28, 2019 · 74 comments
Closed
41 tasks done

features for UL re-miniAOD in 10_6_X #27889

slava77 opened this issue Aug 28, 2019 · 74 comments

Comments

@slava77
Copy link
Contributor

slava77 commented Aug 28, 2019

This is to keep track of features needed for the UL re-miniAOD.
The 2017 will likely happen first.
The target start date is still unclear (it will probably be around October 2019).

As of Aug 7 (after #31072 was merged) the re-miniAOD definitions aiming for UL re-miniAOD should use a run2_miniAOD_UL modifier, if the configuration is to be controlled by a modifier.

[as of Dec 9]

The following features were descoped:

A bit outside of re-miniAOD scope (requires fresh MC/rereco)

@slava77
Copy link
Contributor Author

slava77 commented Aug 28, 2019

assign reconstruction,xpog,operations

@cmsbuild
Copy link
Contributor

New categories assigned: operations,xpog,reconstruction

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

@cmsbuild
Copy link
Contributor

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 Sep 3, 2019

it sounds like there is still time for deploying the fix in the 2018 UL.
I propose to make run2_tau_ul_2018 and run2_tau_ul_2016 modifiers which can be added to Run2_2018 and Run2_2016 eras.
There is not need to propagate these to the master.

@fabiocos @kpedro88 @srimanob this is as a follow up to the discussion today in the ORP.
Please check if this proposal makes sense to you as well.

@peruzzim
Copy link
Contributor

peruzzim commented Sep 4, 2019

it sounds like there is still time for deploying the fix in the 2018 UL.
I propose to make run2_tau_ul_2018 and run2_tau_ul_2016 modifiers which can be added to Run2_2018 and Run2_2016 eras.
There is not need to propagate these to the master.

@fabiocos @kpedro88 @srimanob this is as a follow up to the discussion today in the ORP.
Please check if this proposal makes sense to you as well.

I agree with the above, but please let's keep it active also under the generic devel modifier, so we will remember to add it also to 2017 in a future re-miniAOD of that.

@peruzzim
Copy link
Contributor

peruzzim commented Nov 4, 2019

taking note of #27929 (comment)

@peruzzim
Copy link
Contributor

peruzzim commented Feb 4, 2020

taking note of #28844

@peruzzim
Copy link
Contributor

taking note of #25884

@peruzzim
Copy link
Contributor

also taking note of missing unprefireable bit information in UL miniAOD

The envisaged solution is:

  • run something equivalent to the L1Repack:uGT step, re-emulating ExternalAlgBlock in data with a dedicated customisation
  • adapt NanoAOD configuration accordingly

@rekovic @srimanob

@peruzzim
Copy link
Contributor

assign l1

@cmsbuild
Copy link
Contributor

New categories assigned: l1

@benkrikler,@rekovic you have been requested to review this Pull request/Issue and eventually sign? Thanks

@srimanob
Copy link
Contributor

FYI @ameyer-cern

@peruzzim
Copy link
Contributor

taking note of MUO request for additional tracks to be added in MiniAOD supporting SF derivation without access to AOD

@dntaylor

@slava77
Copy link
Contributor Author

slava77 commented Apr 8, 2020

@danbarto @ahinzmann
I suppose that we also need a backport for #28370

@ahinzmann
Copy link
Contributor

Will need also #29392

@peruzzim
Copy link
Contributor

taking note of #29062

@slava77
Copy link
Contributor Author

slava77 commented Apr 17, 2020

I've updated the issue description #27889 (comment) with the list of features mentioned in the thread.
Please check if something is missing

@slava77
Copy link
Contributor Author

slava77 commented Apr 17, 2020

I missed the PPS plans; added now in the issue description

@jan-kaspar @fabferro

@slava77
Copy link
Contributor Author

slava77 commented Oct 2, 2020

updates from Oct 2:

@slava77
Copy link
Contributor Author

slava77 commented Oct 2, 2020

updates from Oct 2:

* #31510 HCAL slimmed collections  was merged Sep 23, 2020)

* new feature added: #31615 ParticleNet AK4 jet tagger

* info updated with master PR now available ( #31618) for Activate deepDoubleXV2

also, added a note for "produce lowPtGsfEle slimmed collections": Target by October 10 or so.

@slava77
Copy link
Contributor Author

slava77 commented Oct 16, 2020

updates

added

@mariadalfonso
Copy link
Contributor

update

@slava77
Copy link
Contributor Author

slava77 commented Oct 29, 2020

update

* Electron Scale and smearing for UL
  (backport needed for) #31936

this is added now in the description

also updated at the top:

@slava77
Copy link
Contributor Author

slava77 commented Nov 13, 2020

updates:

@slava77
Copy link
Contributor Author

slava77 commented Nov 13, 2020

@lathomas @ahinzmann @kirschen @cms-sw/xpog-l2
please comment on the status of the following items which do not yet have a PR :

@lathomas
Copy link
Contributor

Both are ongoing. 2017 prefiring maps could be ready soon. 2016 will take longer (need UL16 JECs as inputs).
For qgl, it might take some time. This would be nice to have in reMINIAOD but not critical (i.e. I wouldn't delay everything for this).

@slava77
Copy link
Contributor Author

slava77 commented Nov 13, 2020

Both are ongoing. 2017 prefiring maps could be ready soon. 2016 will take longer (need UL16 JECs as inputs).
For qgl, it might take some time. This would be nice to have in reMINIAOD but not critical (i.e. I wouldn't delay everything for this).

Thank you for clarifying.
Do I read your comment correctly that the prefiring maps are still a required feature for remini,
while the qgl is more of a nice to have?

@lathomas
Copy link
Contributor

I don't think that the prefiring maps are such a big deal than they justify delaying reMINIAOD, given that the prefiring weights can be recomputed on top of MINIAOD.

That being said, the timeline is the following:
2017 maps should hopefully come soon (in a week or two).
2016 maps won't be ready before JECs so it will probably take a month (but maybe reMINI of UL16 MC won't have started yet?).

By the way, the prefiring weights are NOT saved in MINIAOD right now. The producer is however used to compute the weights that are stored NANOAOD, and this is why we would care (but then a future reNANO could add them)

@slava77
Copy link
Contributor Author

slava77 commented Nov 13, 2020

@bainbrid
please list the remaining features for low-pt electrons that will not enter #31220 and are planned for separate PRs

@bainbrid
Copy link
Contributor

bainbrid commented Nov 20, 2020

In response to #27889 (comment):

For low-pT electrons:

  • we confirm that we will add code/configuration to perform an energy regression using RECO/AOD inputs
  • we confirm the need for a final ID training (to replace the current placeholder) and associated PR for 11_2 and 10_6.
  • we are considering to embed flags that identify if the pat::Electron originates from a converted photon
  • we are considering to allow updates to the regression and ID using MINIAOD inputs ("MiniAODfromMiniAOD")

@crovelli @cavalari @nancymarinelli

@perrotta
Copy link
Contributor

Additional tracks information for the lostTracks, which was submitted by the tau group in the master with #32242, could be considered for the backport, if it can be done in the shadow of the other pending pull requests (see #32242 (comment))

@perrotta
Copy link
Contributor

In response to #27889 (comment):

For low-pT electrons:

  • we confirm that we will add code/configuration to perform an energy regression using RECO/AOD inputs
  • we confirm the need for a final ID training (to replace the current placeholder) and associated PR for 11_2 and 10_6.
  • we are considering to embed flags that identify if the pat::Electron originates from a converted photon
  • we are considering to allow updates to the regression and ID using MINIAOD inputs ("MiniAODfromMiniAOD")

@crovelli @fcavallo @nancymarinelli

@bainbrid I imagine you were supposed to tag Francesca Cavallari (of whom I don't know the github name, sorry...) instead of Francesca Cavallo (Muon DPG)

@bainbrid
Copy link
Contributor

@crovelli @fcavallo @nancymarinelli

@bainbrid I imagine you were supposed to tag Francesca Cavallari (of whom I don't know the github name, sorry...) instead of Francesca Cavallo (Muon DPG)

Fixed, thanks! @cavalari

@slava77
Copy link
Contributor Author

slava77 commented Nov 27, 2020

updates from today (now in the issue description/list):

@slava77
Copy link
Contributor Author

slava77 commented Dec 9, 2020

updates from near the finish line :

@slava77
Copy link
Contributor Author

slava77 commented Jan 29, 2021

+1

it sounds like we are in a production stage now

@mariadalfonso
Copy link
Contributor

+xpog

indeed production advancing

@srimanob
Copy link
Contributor

@cms-sw/operations-l2 should this issue be closed?

@perrotta
Copy link
Contributor

@cms-sw/operations-l2 should this issue be closed?

I think so

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