Make SkyWcs the default bbox WCS and calculate WCS errors.

dirac-institute / kbmod

KBMOD (Kernel-Based Moving Object Detection)

BSD 2-Clause "Simplified" License

46 stars 14 forks source link

Make SkyWcs the default bbox WCS and calculate WCS errors. #687

Closed DinoBektesevic closed 3 months ago

DinoBektesevic commented 3 months ago

Add wcs error column to the butler standardizer. Make SkyWcs the default way to calculate BBOXes so that we can confidently say that's correct. Rejigger the ordering of the points in the BBOX to match AstroPy convention.

DinoBektesevic commented 3 months ago

Yep, I do math with SkyWcs now so that wasn't mocked. I'm just going to wait for Colin to confirm the new collections actually work before merging cause it's possible I overlooked something. Thanks for the review.