In the TargetList prototype the calc_saturation_dMag function always assumed that the denominator of the integration time used the model from Nemati. But the calc_intTime function treats occulters differently and assumes no speckle noise. That same condition was added to the calc_saturation_dMag function so that the values returned are infinite instead of assuming speckle noise.
In GarrettCompleteness there was an error in the handling of infinite smax values which occur for occulters. Added a check that changes infinite smax values to the max float value.
Type of change
Bug fix (non-breaking change which fixes an issue)
Reference any relevant issues (don't forget the #)
N/A
Checklist before requesting a review
[x] I have verified that all unit tests pass in a clean virtual environment and added new unit tests, as needed
[x] I have run e2eTests and added new test scripts, as needed
[x] I have verified that all docstrings are properly formatted and added new documentation, as needed
Describe your changes
In the TargetList prototype the calc_saturation_dMag function always assumed that the denominator of the integration time used the model from Nemati. But the calc_intTime function treats occulters differently and assumes no speckle noise. That same condition was added to the calc_saturation_dMag function so that the values returned are infinite instead of assuming speckle noise.
In GarrettCompleteness there was an error in the handling of infinite smax values which occur for occulters. Added a check that changes infinite smax values to the max float value.
Type of change
Reference any relevant issues (don't forget the #)
N/A
Checklist before requesting a review
e2eTestsand added new test scripts, as needed