aperiosoftware / astropy

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

Tidy/comment main tiled_compression.py code #55

Closed astrofrog closed 1 year ago

astrofrog commented 1 year ago

The main code for compress_hdu and decompress_hdu has grown organically while trying to solve issues, but we should probably review whether any of the code should live in e.g. the codecs (especially when it relates to only one compression type).

We might want to also consider whether to just return/take arrays from the codecs based on the previous discussion as it looks like other codecs use bytes or arrays as input/output, not actually buffer objects. This would maybe allow buffer_to_array to be subsumed into individual codec classes especially since there is a bunch of algorithm-specific stuff in there.