CMS-HGCAL / cmssw

CMS Offline Software
http://cms-sw.github.io/
Apache License 2.0
3 stars 2 forks source link

WIP: Proposal to structure dense indexer classes #89

Closed pfs closed 9 months ago

pfs commented 10 months ago

The idea is that each indexer class holds the following items

pfs commented 10 months ago

@kerstinlovisa

kerstinlovisa commented 9 months ago

@pfs I'll need to make some changes again to handle the new typecodes for the silicon cells. So just checking: should I translate the typecode to the type numbering we had before type ={0:'F', 1:'T', 2:'B', 3:'L', 4:'R', 5:'5'} for the SoA to avoid storing the string values?

pfs commented 9 months ago

@kerstinlovisa thanks, we can add such decoding utility indeed. But I would propose to keep the internal mapping with the typecode string as is. The classes will know which final index corresponds to each unique typecode without need to really know what it means.