Skip to content

Conversation

cyyever
Copy link
Contributor

@cyyever cyyever commented Sep 24, 2025

What does this PR do?

Python 3.10 has moved Callable to collections.abc

@cyyever cyyever force-pushed the import_Callable branch 3 times, most recently from f0775c6 to 95aef27 Compare September 25, 2025 11:40
@Rocketknight1
Copy link
Member

@cyyever will this break Py3.9? We might pause merging it for a couple of months until Py3.9 reaches EOL in that case.

@cyyever
Copy link
Contributor Author

cyyever commented Sep 25, 2025

@Rocketknight1 We have switched to Py3.10

@Rocketknight1
Copy link
Member

We've switched in the CI, but I don't know if we're intentionally breaking Py3.9 support yet! cc @ydshieh are we officially dropping it now? We might want to update the frontpage README in that case

@cyyever
Copy link
Contributor Author

cyyever commented Sep 25, 2025

@Rocketknight1 Yes, we can delay it.

@cyyever cyyever marked this pull request as draft September 25, 2025 14:31
@cyyever cyyever marked this pull request as ready for review September 30, 2025 13:45
@cyyever
Copy link
Contributor Author

cyyever commented Sep 30, 2025

@Rocketknight1 @ydshieh EOL of 3.9 is Oct 1.

@Rocketknight1
Copy link
Member

I guess we can merge this, then! Can you check the CI and ping me once it's green?

Copy link
Contributor

[For maintainers] Suggested jobs to run (before merge)

run-slow: aimv2, albert, align, altclip, apertus, arcee, aria, audio_spectrogram_transformer, bamba, bart, bert, bert_generation, bigbird_pegasus, biogpt

@ydshieh
Copy link
Collaborator

ydshieh commented Sep 30, 2025

cc @ydshieh are we officially dropping it now?

Not officially yet, and I didn't know it's Oct 1.

I got this search result

https://endoflife.date/python

which says it's Oct 31.

and this

https://devguide.python.org/versions/

only mention Oct but not the day.

I would prefer to wait until it reaches EOL and then drop the support on our side.

@cyyever
Copy link
Contributor Author

cyyever commented Sep 30, 2025

😂 I was told that it is tomorrow...

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