bugGNU Astronomy Utilities - Bugs: bug #53825, NoiseChisel not accounting for...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #53825: NoiseChisel not accounting for fully zero-valued tiles

Submitter:  Mohammad Akhlaghi <makhlaghi>
Submitted:  Thu 03 May 2018 06:07:06 PM UTC
   
 
Category:  NoiseChisel Severity:  3 - Normal
Item Group:  Crash Status:  Fixed
Privacy:  Public Assigned to:  makhlaghi
Open/Closed:  Closed

Thu 03 May 2018 06:28:17 PM UTC, comment #1: 

This bug has been fixed and pushed to the main repo.

This bug was found thanks to a dataset provided by Brandon Kelly.

Mohammad Akhlaghi <makhlaghi>
Group administrator
Thu 03 May 2018 06:07:06 PM UTC, original submission:  

It might happen that the input dataset has zero values for regions with no data: essentially using zero as a mask value. In other detection methods (that have above-Sky thresholds) this isn't an issue. But NoiseChisel's threshold is below the Sky value, so the zero valued pixels will actually be considered as detections. We therefore encourge users to set zero valued pixels to NaN/blank.

When there are tiles with fully zero-valued pixels, they may thus be used for estimating the pseudo-detection Sky and Sky STD and result in zero-valued Sky (and its STD) values. This will later cause the counts-per-second correction (`cpscorr') to be zero and produce `inf' S/N values.

It is thus necessary to also mask tiles that pass all other tests, but produce a standard deviation of 0.

Mohammad Akhlaghi <makhlaghi>
Group administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by makhlaghi (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-05-03 makhlaghi StatusIn Progress Fixed
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-4b48.
    Corresponding source code