astromatic/sextractor

Request: Rice compression support

Opened this issue · 1 comments

teake commented

Related: https://www.astromatic.net/forum/showthread.php?tid=597

At least one pipeline the Euclid SGS relies on a version of SExtractor with support for Rice compression. This version is built from the Roy branch if I'm not mistaken. It would be good if the current processing environment of Euclid could just use the latest and the greatest version of SExtractor (which is a bit more accurate in my testing). Hence this feature request for Rice compression support.

I managed rebase the Roy Branch on top of current master branch, and I'd be happy to put in a PR for this. But I'm curious why this has not been merged yet into master. It is admittedly rough around the edges (e.g. non-optional CFITSIO dependency and a hackish compression detection) and I can try my best to polish these. But are there any other blocking issues with this branch?

Note: from my understanding it is a no-no for the respective Euclid pipeline to first unpack with e.g. funpack and the process with SExtractor, so build-in Rice compression is essential for them.

Hi @teake. Yes the non-optional dependency had me prefer to polish the code before merging. Unfortunately, like for many other things I never found the time to do so. If you are willing to do so, please do. That would be great! Thanks.