Cells and wafers are pairs of integers instead of single integer in new geometry

[clelange]

Sunanda suggested to define wafer/cell as a pair in the ntupliser, for the old geometry the second integer will be 0. This will change the ntuple format.

[rovere]

I'm not sure I follow. If the wafer/cell is returned as a pair in CMSSW, we can still decouple them and leave the format unchanged, no?

[clelange]

Sorry, I wasn't clear: each cell is returned as a pair (either x-y coordinates or u-v), see https://github.com/cms-sw/cmssw/blob/master/DataFormats/ForwardDetId/interface/HGCSiliconDetId.h#L62, and so is each wafer for the new geometries.

BTW, we'd also have to choose u-v or x-y, but they can be converted easily:

int waferX() const { return (-2*waferU()+waferV()); }
int waferY() const { return (2*waferV()); }

[rovere]

Ah ok, this makes sense. I vote to store X,Y in this case.