Skip to content

feat(fits): enable image parsing and example - #31

Merged
ATTron merged 5 commits into
ATTron:mainfrom
arrufat:zignal
Aug 10, 2025
Merged

feat(fits): enable image parsing and example#31
ATTron merged 5 commits into
ATTron:mainfrom
arrufat:zignal

Conversation

@arrufat

@arrufat arrufat commented Aug 7, 2025

Copy link
Copy Markdown
Contributor

Re-enables FITS image parsing using cfitsio and zignal. Replaces zigimg with zignal for image processing. Adds parse_fits_image.zig example.

Re-enables FITS image parsing using `cfitsio` and `zignal`.
Replaces `zigimg` with `zignal` for image processing.
Adds `parse_fits_image.zig` example.
@arrufat

arrufat commented Aug 7, 2025

Copy link
Copy Markdown
Contributor Author

DISCLAIMER: I am the author of zignal
I've also updated cfitsio to work with the latest Zig: allyourcodebase/cfitsio#1

@ATTron

ATTron commented Aug 7, 2025

Copy link
Copy Markdown
Owner

Hey thanks for this! I will take a look at this sometime today! Do you mind also updating the readme to remove the metions i have about the image generation currently not working as well ?

arrufat added 4 commits August 8, 2025 13:00
Renames `parse_fits_image.zig` to `parse_fits_file.zig` and updates its
implementation to explicitly open `test/sample_fits.fits` using
GeneralPurposeAllocator.

Also updates the cfitsio dependency in build.zig.zon.
@arrufat

arrufat commented Aug 8, 2025

Copy link
Copy Markdown
Contributor Author

The latest zig also removed BoundedArray, so I fixed that.

@ATTron
ATTron merged commit a4b3738 into ATTron:main Aug 10, 2025
2 checks passed
@ATTron

ATTron commented Aug 10, 2025

Copy link
Copy Markdown
Owner

thanks for this! happy to have this working again!

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.

2 participants