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

Drop python 3.9 #1541

Merged
merged 1 commit into from
Jan 24, 2025
Merged

Drop python 3.9 #1541

merged 1 commit into from
Jan 24, 2025

Conversation

dachengx
Copy link
Collaborator

@dachengx dachengx commented Jan 24, 2025

Similar to AxFoundation/strax#962

We wanted to upgrade to tensorflow==2.18.0 + keras==3.8.0. This is needed if we want to use higher numpy version (numpy>=2) and numba versions.

We tried in #1505 (comment). But s2_tf_model need also be updated

s2_tf_model = straxen.URLConfig(
to make the transition.

In this PR, we can not release the constraint of numpy.

@dachengx dachengx changed the title Drop python 3.9 and loosen requirement of numpy Drop python 3.9 Jan 24, 2025
@coveralls
Copy link

coveralls commented Jan 24, 2025

Coverage Status

coverage: 89.799% (+0.01%) from 89.787%
when pulling 82a4986 on loosen_numpy
into 242c8b2 on master.

@dachengx dachengx marked this pull request as ready for review January 24, 2025 19:40
@dachengx dachengx merged commit 4519fa7 into master Jan 24, 2025
6 of 7 checks passed
@dachengx dachengx deleted the loosen_numpy branch January 24, 2025 19:51
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.

2 participants