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

Investigate usage of ME0 related code in CMSSW, and possible cleanup of obsolete files #31424

Open
perrotta opened this issue Sep 10, 2020 · 11 comments

Comments

@perrotta
Copy link
Contributor

perrotta commented Sep 10, 2020

There could be some ME0 labelled code in CMSSW which is not in usage any more, having possibly been incorporated in the "standard" GEM code or simply obsolete and never removed.

Muon POG (@ArnabPurohit @trocino) and GEM ( @jshlee @watson-ij) reco contacts are investigating about it

This issue is just to take note of it, and keep track of the advancements

@cmsbuild
Copy link
Contributor

A new Issue was created by @perrotta .

@Dr15Jones, @dpiparo, @silviodonato, @smuzaffar, @makortel, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@perrotta
Copy link
Contributor Author

assign reconstruction

@cmsbuild
Copy link
Contributor

New categories assigned: reconstruction

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

@dildick
Copy link
Contributor

dildick commented Sep 10, 2020

@perrotta While not reconstruction, the ME0 pseudo digis and pseudo segments for the L1-trigger (https://github.com/cms-sw/cmssw/blob/master/L1Trigger/L1TGEM/python/me0TriggerDigis_cff.py#L8) will be phased out when we consolidate GEM and ME0.

@perrotta
Copy link
Contributor Author

perrotta commented Sep 10, 2020

@silviodonato While not reconstruction, the ME0 pseudo digis and pseudo segments for the L1-trigger (https://github.com/cms-sw/cmssw/blob/master/L1Trigger/L1TGEM/python/me0TriggerDigis_cff.py#L8) will be phased out when we consolidate GEM and ME0.

Thank you @dildick . Indeed this will quite likely span several areas, I guess.
I add assignments to L1, and simulation as well

@perrotta
Copy link
Contributor Author

assign l1, simulation

@cmsbuild
Copy link
Contributor

New categories assigned: simulation,l1

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

@dildick
Copy link
Contributor

dildick commented Sep 10, 2020

Yes, many more areas will be affected, including Digitization, Trigger, Reconstruction, Validation and Visualization. Within the GEM group we have a good idea what needs to be done.

@jpata
Copy link
Contributor

jpata commented May 17, 2022

type muon

@cmsbuild cmsbuild added the muon label May 17, 2022
@jpata
Copy link
Contributor

jpata commented May 17, 2022

@cms-sw/muon-pog-l2 @CeliaFernandez @trocino
@cms-sw/gem-dpg-l2 @jshlee @watson-ij

is there any update on this?

@cecilecaillol
Copy link
Contributor

@giovanni-mocellin Can you please followup on that?

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