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

Fixes #608 AssertionError, Multiproc error #610

Closed HannoSpreeuw closed 1 year ago

HannoSpreeuw commented 1 year ago

Fixes #608 : "Daemonic processes are not allowed to have children".

Also replaces the deprecated imp library.

Parts of the source extraction process have been parallellised, to speed things up. For measuring all the islands that constitute sources, Python's multiprocessing has been applied. This was causing an AssertionError if we have method = "multiproc" in pipeline.cfg.

All unit tests pass and trap-manage.py run job on a set of images will work as well.

@AntoniaR I guess this PR should be reviewed after #605 has been merged since that has been pending for a while.

AntoniaR commented 1 year ago

This passes the tests on our system and successfully runs on a dataset.