Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Source Extractor using wrong gain, saturation #9

Open
laldoroty opened this issue Jul 16, 2024 · 1 comment
Open

Source Extractor using wrong gain, saturation #9

laldoroty opened this issue Jul 16, 2024 · 1 comment
Assignees

Comments

@laldoroty
Copy link
Collaborator

After cross convolve, the following warning is thrown:

MeLOn CheckPoint [conv_align_skysub_Roman_TDS_simple_model_R062_37396_8.fits]: Run Python Wrapper of SExtractor!
/hpc/group/cosmology/lna18/sfft/sfft/utils/pyAstroMatic/PYSEx.py:493: UserWarning: MeLOn WARNING [conv_align_skysub_Roman_TDS_simple_model_R062_37396_8.fits]: SExtractor has to use default GAIN = 0!
  warnings.warn('MeLOn WARNING [%s]: %s' %(objname, _warn_message))
/hpc/group/cosmology/lna18/sfft/sfft/utils/pyAstroMatic/PYSEx.py:504: UserWarning: MeLOn WARNING [conv_align_skysub_Roman_TDS_simple_model_R062_37396_8.fits]: SExtractor has to use default SATURATION = 50000.0!
  warnings.warn('MeLOn WARNING [%s]: %s' %(objname, _warn_message))

Should default to gain=1, saturation=100000. The values in this error are default to SFFT. We need our defaults to be 1, 100000.

@laldoroty laldoroty self-assigned this Jul 16, 2024
laldoroty added a commit that referenced this issue Jul 19, 2024
…oadd_img(). Set correct default gain and saturation levels in PYSEx.PS(), addressing issue #9.
@laldoroty
Copy link
Collaborator Author

I updated my forked SFFT package (https://github.com/laldoroty/sfft) to allow choosing gain=1, saturation=100000, but I think we should leave this open and figure out if we need real gain/saturation values in headers. Or if these values are always correct because it's a sim...?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant