aperiosoftware / astropy

Repository for the Astropy core package
www.astropy.org
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Implement (De)Quantization algorithm #14

Closed astrofrog closed 1 year ago

astrofrog commented 1 year ago

The quantize.c file in cfitsio is ~4000 lines long and depends on other parts of cfitsio, so might not be easy to use directly. I think we can't use the numcodecs Quantize codec because the FITS standard (section 10.2) describes a set of very specific algorithms to use for quantization.

However, I think the algorithms might be simple enough that we can re-implement ourselves (perhaps even with just Python + Numpy). One thing we will need is the list of 10000 random numbers mentioned in 10.2 of the FITS standard, but we could potentially even store the hard-coded values as this should only take 40kb.

github-actions[bot] commented 1 year ago

Welcome to Astropy 👋 and thank you for your first issue!

A project member will respond to you as soon as possible; in the meantime, please double-check the guidelines for submitting issues and make sure you've provided the requested details.

GitHub issues in the Astropy repository are used to track bug reports and feature requests; If your issue poses a question about how to use Astropy, please instead raise your question in the Astropy Discourse user forum and close this issue.

If you feel that this issue has not been responded to in a timely manner, please send a message directly to the development mailing list. If the issue is urgent or sensitive in nature (e.g., a security vulnerability) please send an e-mail directly to the private e-mail feedback@astropy.org.

astrofrog commented 1 year ago

Done in #31