search

keflavich / cube-line-extractor

4 stars 1 forks source link

Feature Request: Add noise and masking information to FITS headers #10

Closed jmangum closed 2 years ago

jmangum commented 7 years ago

Since the RMS noise and masking parameters used to derive the moment maps from CubeLineMoment are often useful to know when using these measurements in an analysis of their results, I wonder if it would be useful to add this information to the moment FITS headers?

keflavich commented 7 years ago

Sure, that's easy to do. Just add header parameters with header['MASKLEV'] = mask_level or something along those lines.

jmangum commented 7 years ago

ok. Let me try to take this one on.

jmangum commented 2 years ago

Should these be added as header keywords or comment or history entries?

keflavich commented 2 years ago

Either way is fine; keywords are easier to access later. You can do thinks like: header['MASKLEV'] = (mask_level, 'Threshold in Jy used to create the mask cube') if you want a / comment added to the header entry

jmangum commented 2 years ago

ok. Will use header.