Skip to content

Conversation

@arrufat
Copy link
Contributor

@arrufat arrufat commented Aug 7, 2025

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
Copy link
Contributor Author

arrufat commented Aug 7, 2025

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

@ATTron
Copy link
Owner

ATTron commented Aug 7, 2025

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
Copy link
Contributor Author

arrufat commented Aug 8, 2025

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
Copy link
Owner

ATTron commented Aug 10, 2025

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