Skip to content

platform independent script to verify sha256 checksums #1203

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

Merged
merged 7 commits into from
May 3, 2023

Conversation

KASR
Copy link
Contributor

@KASR KASR commented Apr 27, 2023

feat: add python script to verify the file hashes, the goal is to be platform independent

the output is a table with the filename, valid checksum (if the hash matches), and file missing (if the file is not found).

KASR added 2 commits April 27, 2023 09:47
Added Python script for verifying SHA256 checksums of files in a directory, which can run on multiple platforms. Improved the formatting of the output results for better readability.
update to the readme for improved readability and to explain the usage of the python checksum verification script
I've extended the script based on suggestions by @prusnak

The script now checks the available RAM, is there is enough to check the file at once it will do so. If not the file is read in chunks.
@prusnak
Copy link
Collaborator

prusnak commented Apr 27, 2023

There is no reason to overengineer the script with psutil. There is no reason not to read in chunks. Please keep the script simple.

KASR added 2 commits April 27, 2023 16:51
small change so that the available ram is checked and not the total ram
… is available

based on suggestions from @prusnak i removed the part of the code that checks whether the user had enough ram to read the entire model at once. the file is now always read in chunks.
@KASR
Copy link
Contributor Author

KASR commented Apr 27, 2023

There is no reason to overengineer the script with psutil. There is no reason not to read in chunks. Please keep the script simple.

ok I've updated the script accordingly. the file is now always read in chunks.

@KASR KASR requested a review from prusnak April 27, 2023 15:06
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.

3 participants