cta-observatory / cta-lstchain

LST prototype testbench chain
https://cta-observatory.github.io/cta-lstchain/
BSD 3-Clause "New" or "Revised" License
24 stars 77 forks source link

Actually compute pixel distance to camera center in Long-Term DL1 check #1074

Open maxnoe opened 1 year ago

maxnoe commented 1 year ago

One plot shows cosmic pulses vs. pixel-id, implying that one should see a falling trend with the pixel id.

This somewhat works as the pixel id is sorted by the spiral pattern but there is also a "sub-pattern" due to the hexagonal spirals.

We should just compute the pixel's distance to the camera center as r = np.sqrt(cam.pix_x**2 + cam.pix_y**2) and use that for plotting

moralejo commented 1 year ago

The purpose of the plots is just to facilitate finding the value of a quantity for a given pixel id, if needed. Given the spiral numbering they also show roughly the dependency of the quantity vs. distance to camera center, but that is just a side effect. Using the true distance would still be a bit misleading for outermost pixels, for a given distance-to-center some pixels are at the edge of the camera while others are still well inside (towards the corners).

maxnoe commented 1 year ago

There is a border that shows the expected interval, which heavily implies there is a connection to the distance...

moralejo commented 1 year ago

Yes, those are just rough limits which are relaxed near the border because of edge effects.

And indeed, the limits in those plots are only valid for LST's spiral pixel numbering which more or less correlates with distance. Certainly for a generic pixel numbering this would have to be modified.