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

Overview of implementations to consume HCalPFCuts from DB #43312

Open
malbouis opened this issue Nov 17, 2023 · 7 comments
Open

Overview of implementations to consume HCalPFCuts from DB #43312

malbouis opened this issue Nov 17, 2023 · 7 comments

Comments

@malbouis
Copy link
Contributor

This issue is to keep track of all the developments towards having the HCalPFCuts consumed from the GT.

So far, we have the following:

Issues opened:

It would be nice to identify all the needed CMSSW changes in order to consume the HcalPFCuts from the GT. Will come back to this a later stage, but please, all, feel free to add any additional information to this issue.

FYI: @cms-sw/alca-l2 , @cms-sw/hlt-l2 , @cms-sw/pf-l2 , @cms-sw/egamma-pog-l2 , @cms-sw/orp-l2 , @cms-sw/ppd-l2

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 17, 2023

A new Issue was created by @malbouis Helena Malbouisson.

@sextonkennedy, @makortel, @rappoccio, @Dr15Jones, @smuzaffar, @antoniovilela can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@mmusich
Copy link
Contributor

mmusich commented Nov 17, 2023

type hcal, pf, egamma

@mmusich
Copy link
Contributor

mmusich commented Nov 17, 2023

assign alca, hlt, reconstruction

@cmsbuild
Copy link
Contributor

New categories assigned: alca,hlt,reconstruction

@Martin-Grunewald,@mmusich,@jfernan2,@mandrenguyen,@saumyaphor4252,@perrotta,@consuegs you have been requested to review this Pull request/Issue and eventually sign? Thanks

@perrotta
Copy link
Contributor

A search made on lxr some time ago ended up exactly with the same packages already mentioned in this issue, i.e. the ones already addressed by the open PRs and the RecoLocalCalo/CaloTowers.
Just in case there are other hidden dependencies in our code on those HCalPFCuts, let me tag here also all other possible relevant POGs, which are also invited to look in their own code for such possible dependencies (and provide the needed updates, if so): @cms-sw/muon-pog-l2 @cms-sw/tau-pog-l2 @cms-sw/jetmet-pog-l2

@swagata87
Copy link
Contributor

Alpaka implementations to be adapted to read HCalPFCuts from DB: #43130 and #42807

Done in #43574
(thanks @jsamudio)

@mmusich
Copy link
Contributor

mmusich commented Jan 30, 2024

@RSalvatico pointed me to an issue in #43164, see #43164 (comment)

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

5 participants