In tests/pipeline/test_alerting.py , the test test_alerts_with_previous is currently marked to be skipped. Reason: I found images in the DECat pipeline that used the reference were already using in tests (so I didn't have to rewrite a whole bunch of other tests if we changed our test DECam images) that had detections. However, SeeChange is not finding them. It's possible that the detections were not real (though I'm skeptical; they look like legitimate quasars) and the DECat pipeline detected them where it shouldn't, and zogy is properly missing it. It's also possible that zogy is missing real things.
Once more investigation into what we're catching in subtraction is done, come back and revisit this test.
In
tests/pipeline/test_alerting.py, the testtest_alerts_with_previousis currently marked to be skipped. Reason: I found images in the DECat pipeline that used the reference were already using in tests (so I didn't have to rewrite a whole bunch of other tests if we changed our test DECam images) that had detections. However, SeeChange is not finding them. It's possible that the detections were not real (though I'm skeptical; they look like legitimate quasars) and the DECat pipeline detected them where it shouldn't, and zogy is properly missing it. It's also possible that zogy is missing real things.Once more investigation into what we're catching in subtraction is done, come back and revisit this test.