Thanks to visit codestin.com
Credit goes to github.com

Skip to content

TST: Add tests for savez and savez_compressed reproducibility #30182

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

TST: Add tests for savez and savez_compressed reproducibility #30182

wants to merge 1 commit into from
+16 −0

Conversation

@tovrstra
Copy link
Contributor

@tovrstra tovrstra commented Nov 9, 2025

Closes #9439

@meawoppl Can you check if this is ok for you?

@tovrstra
Copy link
Contributor Author

tovrstra commented Nov 9, 2025 •
edited
Loading

All failing tests seem to be unrelated to the proposed test. This is a representative error message:

Attempt to get headers for http://downloads.sourceforge.net/project/pkgconfiglite/0.28-1/pkg-config-lite-0.28-1_bin-win32.zip failed.
  The remote file either doesn't exist, is unauthorized, or is forbidden for url 'http://downloads.sourceforge.net/project/pkgconfiglite/0.28-1/pkg-config-lite-0.28-1_bin-win32.zip'. Exception calling "GetResponse" with "0" argument(s): "The remote server returned an error: (403) Forbidden."
Downloading pkgconfiglite 64 bit
  from 'http://downloads.sourceforge.net/project/pkgconfiglite/0.28-1/pkg-config-lite-0.28-1_bin-win32.zip'
ERROR: The remote file either doesn't exist, is unauthorized, or is forbidden for url 'http://downloads.sourceforge.net/project/pkgconfiglite/0.28-1/pkg-config-lite-0.28-1_bin-win32.zip'. Exception calling "GetResponse" with "0" argument(s): "The remote server returned an error: (403) Forbidden."
The install of pkgconfiglite was NOT successful.

The download from sourceforge seems to cause problems.

@charris
Copy link
Member

charris commented Nov 11, 2025

Close/reopen

@charris charris closed this Nov 11, 2025
@charris charris reopened this Nov 11, 2025
@meawoppl
Copy link

meawoppl commented Nov 13, 2025 •
edited
Loading

This fixes the problems I reported...8 years ago?

NGL, I don't even remember the original context in which it percolated, so ... thank you!

@tovrstra
Copy link
Contributor Author

tovrstra commented Nov 14, 2025 •
edited
Loading

You're welcome. Better late than never, but note that it is just a test to verify deterministic behavior. The fix itself is way older.

I was looking into this because I'd be interested in making the NPZ files also platform-independent, which goes just a bit further. (One step at a time.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

05 - Testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

BUG: savez() is not deterministic

3 participants

@tovrstra @charris @meawoppl