cta-observatory/nectarchain

NectarCam geometry

sizun opened this issue · 11 comments

sizun commented

Hello @FrancaCassol

I am confused by the CameraDisplay obtained with the current geometry file NectarCam-002.camgeom.fits.gz, using either the CameraFrame or the EngineeringCameraFrame coordinate system:

ctapipe_camera_display

The displays do not correspond to the view I am used too e.g. looking at the camera from the dish, x points right, y points up (which according to the doc of these classes is the MAGIC/FACT convention the engineering frame should provide?). In this frame, for NectarCam, the pixel numbers should increase like this:

nectarcam_pixel_numbers_xyCAM

Hi @sizun,

sorry I am not sure to understand the question ( these geometries are always rather confusing for me too). Is it not that you should look at the pixel index from behind the camera not from the front?

I think the NectarCam-002 file seems to have the X and Y axes swapped, at least with respect to the simulation version. I think in the EngineeringCameraFrame, it should look like the bottom figure in Patrick's digram (but right now it is rotated or flipped 90 deg).  That will cause problems in the reconstruction later also.

In the simulations (prod3) it looks ok, except that the pixel numbering is obviously not correct

I would say that I see a swap left to right of the x axis (the pixel zero is on the right instead of the left)...

Just to be sure, the EngineeringCameraFrame should be like the figure of Patrick?

sizun commented

In the simulations (prod3) it looks ok, except that the pixel numbering is obviously not correct

The pixel numbering was discussed with the MC group (see issue #22534) but I am not sure from which prod it is supposed to be taken into account.

If I swap both axis I get this geometry (where I highlight the first 3 pixels). There is still a top-bottom problem?

geometry_003

sizun commented

If I swap both axis I get this geometry (where I highlight the first 3 pixels).

With EngineeringCameraFrame?

There is still a top-bottom problem?

Yes, if I understand the coordinate system correctly.

This what I have with the two frames.
@kosack what do you think about that?

geometry_003

sizun commented

I made a geometry 004 that leads to the displays I expect (by editing pix_x, pix_y and PIX_ROT):

ctapipe_camgeom_004

What do you think? I highlighted pixels 0,1,2,441,927.

Hi,
it seems good to me, but I leave @kosack to judge, in the case it is ok, please call it 003 and PR in ctapipe-extra with the same naming convention of the 002. I will then modify the nectarcam io package, in order to take this version as default.