CARTAvis / carta-frontend

Source code repository for the frontend component of CARTA, a new visualization tool designed for the ALMA, the VLA and the SKA pathfinders.
https://cartavis.github.io/
GNU General Public License v3.0
19 stars 10 forks source link

no wcs grid for DES fits image #2386

Open kswang1029 opened 4 months ago

kswang1029 commented 4 months ago

Describe the bug This is a user bug report. If we load the image

https://drive.google.com/file/d/1bDbI9-DV-nAssT5bJslAvMOdHWaO7qhJ/view?usp=drive_web

we do not see the wcs grid rendered properly. The console shows errors as

Screenshot 2024-06-01 at 18 24 06

line277: plot = astPlot(wcsinfo, gbox, pbox, args);

If we load the image in ds9 and save as a slice in fits again, then the wcs grid can be rendered successfully in CARTA. For some reason, this trick regulates the fits header.

Let's see if a fix can be applied to handle this case (or not, ie headers fault)

To Reproduce Steps to reproduce the behavior:

  1. download the example image https://drive.google.com/file/d/1bDbI9-DV-nAssT5bJslAvMOdHWaO7qhJ/view?usp=drive_web
  2. load it in CARTA and we won't see wcs grid rendered properly.
  3. load the image in ds9 and save as a slice
  4. load the ds9-processed fits image in CARTA and we will see the wcs grid rendered properly.

Expected behavior can handle such fits header

Platform info (please complete the following information):

loveluthien commented 4 months ago

The problem comes from the file name. If renaming the file with a simpler name, wcs grid works. No need to use ds9 to save it as a new file. It seems that AST has some difficulties to handle complex file name.

kswang1029 commented 4 months ago

The problem comes from the file name. If renaming the file with a simpler name, wcs grid works. No need to use ds9 to save it as a new file. It seems that AST has some difficulties to handle complex file name.

Interesting. Any idea on which char causing the problem?

kswang1029 commented 4 months ago

The problem comes from the file name. If renaming the file with a simpler name, wcs grid works. No need to use ds9 to save it as a new file. It seems that AST has some difficulties to handle complex file name.

Interesting. Any idea on which char causing the problem?

I guess it is "=" or ","?

loveluthien commented 4 months ago

@kswang1029 yes, it is ",".