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

Add CI for Python 3.13 and update dev envs #2518

Merged
merged 1 commit into from
Sep 26, 2024

Conversation

dhomeier
Copy link
Collaborator

@dhomeier dhomeier commented Sep 23, 2024

Description

Introducing test jobs for currently 3.13rc2 and pushing the py310-test-dev envs, which are now stuck at Astropy 6.1.3, to higher Python versions.

Non-dev job for py313 is failing on missing libhdf5-dev when trying to build h5py from source; don't know how to install the apt for just a specific env.
I have down-versioned the windows-dev job here so it runs with astropy 6.1.3; upgrading to py312 produced the same data_factory failures

E AssertionError: assert <function astropy_tabular_data at 0x0000020F8CEE2C00> is <function img_data at 0x0000020F8CEE8FE0>

but the also occur with the non-dev Windows job, so it appears to be a Python 3.11+ issue on that platform.
As far as I can tell this is incorrectly identifying the .png file as readable as an astropy_tabular_data "Catalog".
@pllim any ideas what could have happened there?

Addresses #2517

@dhomeier dhomeier added no-changelog-entry-needed Skipped in Changelog generation testing labels Sep 23, 2024
@dhomeier dhomeier force-pushed the ci-py312 branch 3 times, most recently from 414f93f to 8ab890e Compare September 23, 2024 22:15
@pllim
Copy link
Contributor

pllim commented Sep 24, 2024

Looks like it tests some sort of third-party PNG loader, so I am not sure. What is this actually testing? That df.find_factory(fname) must be the same object as df.img_data? Perhaps somehow a copy is made on Windows, breaking that assumption?

@pllim
Copy link
Contributor

pllim commented Sep 24, 2024

Did this test ever pass before on Windows?

@dhomeier
Copy link
Collaborator Author

It did, and does for Python <=3.10. Turned out to be a locale/encoding issue hopefully solved in #2519.

@astrofrog astrofrog merged commit 7b4c073 into glue-viz:main Sep 26, 2024
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog-entry-needed Skipped in Changelog generation testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants