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

Drizzlepac/HAP: Begin testing with python 3.13 #1930

Open
stscijgbot-hstdp opened this issue Jan 14, 2025 · 2 comments · May be fixed by #1936
Open

Drizzlepac/HAP: Begin testing with python 3.13 #1930

stscijgbot-hstdp opened this issue Jan 14, 2025 · 2 comments · May be fixed by #1936

Comments

@stscijgbot-hstdp
Copy link
Collaborator

Issue HLA-1412 was created on JIRA by Steve Goldman:

Begin testing with python 3.13, to eventually transition to python 3.13. 

@s-goldman s-goldman linked a pull request Jan 21, 2025 that will close this issue
4 tasks
@stscijgbot-hstdp
Copy link
Collaborator Author

Comment by Steve Goldman on JIRA:

While we are now testing with python 3.13, I am now working on trying to get the tests to pass. 

An issue that I've run into is that there is a dependency (stregion) that has the python version pinned to less than python 3.13.  It looks like it was originally pinned because things were failing. 

While the tests appear to work locally, both on linux and mac. The tests fail for the python 3.13 test. When we do another test with all the dependencies forced to most recent version (devdep), the tests result in a c compilation issue (cdriz) which doesn't seem to make sense. We suspect that there is a (uv) installation issue that is not being raised in the logs. 

I've submitted a PR to remove the stregion pin, and will continue trying to diagnose the issue. 

@stscijgbot-hstdp
Copy link
Collaborator Author

stscijgbot-hstdp commented Jan 27, 2025

Comment by Steve Goldman on JIRA:

There are now two PR out for testing with python 3.13. We now have the github action tests that we run on drizzlepac as well as the new regression testing suit. We may be able to get rid of the drizzlepac tests, but for the meantime, I have updated the github action files for both. 

Code for completing this task is in Drizzlepac PR #⁠1936 and spacetelescope/regressiontest PR #⁠191.

 

Another ticket will be created to get the tests to pass. 

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 a pull request may close this issue.

1 participant