roman-corgi / corgidrp

Data Reduction Pipeline for the Roman Coronagraph Instrument
4 stars 2 forks source link

cosmic ray masking update #91

Closed kjl0025 closed 22 hours ago

kjl0025 commented 1 month ago

The masking of cosmic rays using the GSW tool for this was not sufficient to create good noise maps (arrays of dark current, CIC, and FPN per pixel in the image area). It has since been updated and delivered and does lead to good-quality noise maps. Need to update the cosmic-ray masking function accordingly.

maxwellmb commented 3 weeks ago

This is related to Issue #72, so I'm going to close this one and leave that one open.

mygouf commented 1 week ago

@kjl0025 I see you are still assigned to this issue, are you planning to work on this still or would you need help with this?

kjl0025 commented 1 week ago

Ah yes, I did instinctively assign myself. :) However, I was thinking that once you (or someone) pulls in the latest cosmic ray code on corgi_iit_drp, perhaps Ell could implement the changes since she did the initial port of the cosmic code? I could do it, but I am running low on funding for this work.