askap-vast / vast-pipeline

This repository holds the code of the Radio Transient detection pipeline for the VAST project.
https://vast-survey.org/vast-pipeline/
MIT License
8 stars 3 forks source link

Correctly handle compressed fits files #694

Closed ddobie closed 9 months ago

ddobie commented 1 year ago

Fix #693.

ddobie commented 1 year ago

@ajstewart would be good to get your eyes on this before I go further and update docs etc

ajstewart commented 1 year ago

This seems ok to me so far!

ddobie commented 10 months ago

I think this is good to go. I haven't added any tests for the open_fits function because there aren't any for the rest of the utils functions. If it's broken then almost every other test should fail (which was happening earlier this morning) so it's probably fine. Happy to add tests if you think it's necessary though.