search

TESScience / SPyFFI

Spiffy Python for TESS Full Frame Images
MIT License
6 stars 7 forks source link

Modify saturation values in SPyFFI #50

Open zkbt opened 7 years ago

zkbt commented 7 years ago

Modify the saturation levels to be 200,000 electrons per individual read, to be closer to lab tests.

While saturation should now fall under the purview of HTTM, it would be nice to preserve the capability for SPyFFI to produce perfectly calibrated imaging data. To so, we should leave in the option for saturate stars.

xcthulhu commented 7 years ago

Looks like the header is getting set here, to the 'SATURATE' key

https://github.com/TESScience/SPyFFI/blob/4eab9ce4b1ca6625f9f7fc2cb174044cd03e1287/CCD.py#L157

In HTTM, I believe this may be referred to as the full well parameter

https://github.com/TESScience/httm/blob/595528ece1c7f85f8daa536e8da3cc79cd3b1347/httm/data_structures/metadata.py#L109

As you can see, it is set to 170000.

I will make sure that HTTM will look for SATURATE if its around to use in it's blooming model.