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

Investigate enabling validate_checksum as default True #1784

Open
braingram opened this issue Apr 26, 2024 · 0 comments
Open

Investigate enabling validate_checksum as default True #1784

braingram opened this issue Apr 26, 2024 · 0 comments
Milestone

Comments

@braingram
Copy link
Contributor

Description of the problem

To address issues of data corruption (like the pre-commit whitespace stripping that held up: spacetelescope/romancal#1161) investigate enabling validate_checksum as default True.

If not already the behavior (I don't recall at the moment) the validation could occur only when the block is read (so a file opened with lazy_load=True and corrupt data would not raise an error until the corrupt array data is accessed). This may require some changes to how validate_checksum behaves and documentation updates. Depending on the impact of the change a deprecation period could be considered.

@braingram braingram added this to the 3.2.0 milestone Apr 26, 2024
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

No branches or pull requests

1 participant