search

transientskp / tkp

A transients-discovery pipeline for astronomical image-based surveys
http://docs.transientskp.org/
BSD 2-Clause "Simplified" License
19 stars 14 forks source link

Add initial pytest action for github cicd #618

Closed tmillenaar closed 1 week ago

tmillenaar commented 2 months ago

This PR adds a workflow to run the automated tests. It is ran whenever a new commit is pushed and in the current state takes about 7 minutes.

Running the tests is somewhat involved because: 1) A PostgreSQL database needs to be set up to run the tests 2) A casacore container is required for lookup tables 3) The test data is located in a different repository: https://github.com/transientskp/trap-test-data

Item 2) can be simplified in the near future, because the tables we need for TraP are included in the python-casacore binary since the following PR was merged: https://github.com/casacore/python-casacore/pull/260 However, to make use of this we do need to wait for a new python-casacore release. This would mean we would be able to pip install python-casacore and we would not have to run in the casacore container anymore.

HannoSpreeuw commented 1 month ago

I have been scrolling through your changes....seems like a nice cleanup.