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

[HGCAL/TICL] define/assign ticlIteration for HFNose and default-constructed tracksters #34971

Open
slava77 opened this issue Aug 21, 2021 · 4 comments

Comments

@slava77
Copy link
Contributor

slava77 commented Aug 21, 2021

#34962 :

#34605 is an example of using iternames which can not be decoded by RecoHGCal/TICL/plugins/TrackstersProducer.cc. E.g. ticlTrackstersHFNoseEM uses EMn.

Either a new iteration should be defined in the Trackster::IterationIndex enum, or the TrackstersProducer should decode to an existing enum value.

Also, in #34858 the Trackster::iterationIndex_ was default-initialized to 0, which can be confused with EM. Perhaps a new enum UNKNOWN or INVALID should be defined.

@cms-sw/hgcal-dpg-l2

@slava77
Copy link
Contributor Author

slava77 commented Aug 21, 2021

assign reconstruction,hgcal-dpg

@cmsbuild
Copy link
Contributor

A new Issue was created by @slava77 Slava Krutelyov.

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

cms-bot commands are listed here

@cmsbuild
Copy link
Contributor

New categories assigned: hgcal-dpg,reconstruction

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

@jpata
Copy link
Contributor

jpata commented May 5, 2022

type hgcal

@cmsbuild cmsbuild added the hgcal label May 5, 2022
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

3 participants