You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In looking through the code for the makemask function in calib.py, I noticed what may be a small cut-and-paste bug. In line 297 (as seen by looking at the file on github.com using a web browser) the flat_stats variable is defined by a call to stats.sigma_clipped_stats. I think that this should give statistics for the flats, but the input list is img_dk rather than img_fl, as I would expect and as is used in the else portion of this if/else structure. Perhaps this bug never causes problems if people are using up-to-date versions of astropy, but I thought I should bring it to your attention nonetheless.
The text was updated successfully, but these errors were encountered:
In looking through the code for the makemask function in calib.py, I noticed what may be a small cut-and-paste bug. In line 297 (as seen by looking at the file on github.com using a web browser) the flat_stats variable is defined by a call to stats.sigma_clipped_stats. I think that this should give statistics for the flats, but the input list is
img_dk
rather thanimg_fl
, as I would expect and as is used in theelse
portion of thisif/else
structure. Perhaps this bug never causes problems if people are using up-to-date versions of astropy, but I thought I should bring it to your attention nonetheless.The text was updated successfully, but these errors were encountered: