gbrammer / grizli

Grizli: The "Grism redshift and line" analysis software
MIT License
69 stars 51 forks source link

Change default blot stepsize to use WCS transform #248

Closed gbrammer closed 1 month ago

gbrammer commented 1 month ago

The WCS mapping in grizli.utils.blot_nearest_exact using astdrodrizzle.cdriz.DefaultWCSMapping seems to cause slight errors in the pixel transformations, so set the default stepsize=-1 in various places to fall back to the nominal WCS transformation out_wcs.all_pix2world > in_wcs.all_world2pix.

The most notable place where the issue affects the output was in grizli.prep.subtract_visit_angle_averages, where the blotted angle averages didn't line up correctly.