dirac-institute / kbmod

KBMOD (Kernel-Based Moving Object Detection)
BSD 2-Clause "Simplified" License
46 stars 14 forks source link

Add more exhaustive tests for the standardizer values. #672

Closed DinoBektesevic closed 3 months ago

DinoBektesevic commented 3 months ago

The current standardizers are being tested in the sense that they will not error out throughout the codepath, but no exhaustive tests are being performed that they actually do anything to the data they are given.

This was done intentionally, since we knew they will have to change in the future and because creating data is really hard. It would be good to change this and add more exhaustive tests. See also https://github.com/dirac-institute/kbmod/pull/668#discussion_r1690140038