FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

MNT: Upgrade to CFITSIO 4.7.0 by pllim · Pull Request #20255 · astropy/astropy · GitHub

/ astropy Public

MNT: Upgrade to CFITSIO 4.7.0 - #20255

Draft
pllim wants to merge 2 commits into
astropy:mainfrom
pllim:cfitsio-4.7.0
ConversationCommits (2)ChecksFiles changed
Draft

MNT: Upgrade to CFITSIO 4.7.0#20255
pllim wants to merge 2 commits into
astropy:mainfrom
pllim:cfitsio-4.7.0

Conversation

pllim commented Aug 18, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

New CFITSIO release found.

Version: 4.7.0 (Aug 2026)

Change log

Version 4.7.0 - Aug 2026

  • This release includes patches to security vulnerabilities. We strongly encourage this upgrade, particularly for those running CFITSIO in web accessible applications.

    Our thanks to A. Denkiewicz and Z. Bugaud for their extensive help with this.

  • New test framework added, containing a collection of unit tests. Our thanks to W. Pursell for contributing this.

  • Bug fix for parsing octal integer constants in expressions. Our thanks to Cruzzil_Code for submitting this.

  • Enhanced output from uncompress2mem_from_mem function.

  • Appended 'fits_' prefix to stream driver function names to reduce chances of potential symbol name conflicts with outside libraries. Our thanks to rtobar for submitting this.

(For complete change log information, see https://heasarc.gsfc.nasa.gov/FTP/software/fitsio/c/docs/changes.txt .)

  • By checking this box, the PR author has requested that maintainers do NOT use the "Squash and Merge" button. Maintainers should respect this when possible; however, the final decision is at the discretion of the maintainer that merges the PR.

pllim added this to the v8.1.0 milestone Aug 18, 2026
pllim added io.fits external PRs and issues related to external packages vendored with Astropy (astropy.extern) Extra CI Run cron CI as part of PR Build all wheels Run all the wheel builds rather than just a selection labels Aug 18, 2026

github-actions Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Thank you for your contribution to Astropy! 🌌 This checklist is meant to remind the package maintainers who will review this pull request of some common things to look for.

  • Do the proposed changes actually accomplish desired goals?
  • Do the proposed changes follow the Astropy coding guidelines?
  • Are tests added/updated as required? If so, do they follow the Astropy testing guidelines?
  • Are docs added/updated as required? If so, do they follow the Astropy documentation guidelines?
  • Is rebase and/or squash necessary? If so, please provide the author with appropriate instructions. Also see instructions for rebase and squash.
  • Did the CI pass? If no, are the failures related? If you need to run daily and weekly cron jobs as part of the PR, please apply the "Extra CI" label. Codestyle issues can be fixed by the bot.
  • Is a change log needed? If yes, did the change log check pass? If no, add the "no-changelog-entry-needed" label. If this is a manual backport, use the "skip-changelog-checks" label unless special changelog handling is necessary.
  • Is this a big PR that makes a "What's new?" entry worthwhile and if so, is (1) a "what's new" entry included in this PR and (2) the "whatsnew-needed" label applied?
  • At the time of adding the milestone, if the milestone set requires a backport to release branch(es), apply the appropriate "backport-X.Y.x" label(s) before merge.

pllim commented Aug 18, 2026
View reviewed changes
Comment thread astropy/io/fits/hdu/compressed/src/compression.c
decompressed_values = (int *)calloc(tilesize, sizeof(int));

pl_l2pi(compressed_values, 1, decompressed_values, tilesize);
pl_l2pi(compressed_values, tilesize, 1, decompressed_values, tilesize);

pllim Aug 18, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Pretty sure this is wrong. Do we need to add extra stuff to make it look like the RICE compression codec? I am afraid to change the code too much by myself. @saimn or @astrofrog ? 🙏

pllim mentioned this pull request Aug 18, 2026
Closed
astrofrog self-assigned this Aug 19, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

saimn Awaiting requested review from saimn saimn will be requested when the pull request is marked ready for review saimn is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

astrofrog

Labels

Build all wheels Run all the wheel builds rather than just a selection external PRs and issues related to external packages vendored with Astropy (astropy.extern) Extra CI Run cron CI as part of PR io.fits

Projects

None yet

Milestone

v8.1.0

Development

Successfully merging this pull request may close these issues.

2 participants

Back | FazBrowse Home | New Git URL