Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bintable data better support, tile compressed image (gzip and rice), additional things #20

Merged
merged 36 commits into from
Feb 7, 2025

Conversation

bmatthieu3
Copy link
Collaborator

This PR contains:

  • A better support for the bintable extension parsing with variable length array support (seeking into the heap associated to a binary table)
  • W.I.P: tile compressed image convention. Only GZIP and RICE compression algorithm will be supported. For the moment only tiles with ZBITPIX refering to U8 (byte), I16 (short) and I32 (integer) is supported. The quantization method for handling floating point tiles is not done in that PR
  • Jumping to the next HDU uses the Seek trait (optimization)
  • On image extensions: it is possible from the ImageData object to only seek for a specific pixel
  • Introducing of the log crate thanks to the reference given by Log parsing errors to stderr #18 . Logging is used here for warning the user if the parsing did not find some cards. This should be rebase on the work of Comments, continued, strings, card as enum, etc #16

…cking if its internal reader refers to a gz file or nto
…r for retrieving the variable array content in a bintable, gzip compression case w.r.t to the tile compressed image convention
…thm, add links to CFITSIO license in the README for porting its c code into rust
…coding, relax continuation parsing with respect to 4.2.1.2, make the test pass
@bmatthieu3 bmatthieu3 mentioned this pull request Feb 7, 2025
@bmatthieu3 bmatthieu3 merged commit 90b115b into master Feb 7, 2025
1 check passed
@bmatthieu3 bmatthieu3 deleted the bintable branch February 7, 2025 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant