Don't allow the MWS_FAINT targets to affect the state of any secondaries.

desihub / desitarget

DESI Targeting

BSD 3-Clause "New" or "Revised" License

18 stars 23 forks source link

Don't allow the MWS_FAINT targets to affect the state of any secondaries. #746

Closed geordie666 closed 3 years ago

geordie666 commented 3 years ago

This PR prevents the MWS_FAINT targets from merging with and updating the state of secondary targets for the Main Survey. This proved necessary to preserve the original state of secondary targets, as we didn't start the survey with the MWS_FAINT target class.

This is a short PR with a specific focus, so I'll merge it once tests pass.

coveralls commented 3 years ago

Coverage decreased (-0.05%) to 58.748% when pulling a893f114abd673abc2ecf2e0344671f181007183 on ADM-nosec-MWF into 08a902f6bfaee7289ce132de98e66462c725a842 on master.