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

Chore: add python 313 #1157

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Chore: add python 313 #1157

wants to merge 6 commits into from

Conversation

JanEricNitschkeAA
Copy link
Contributor

Description

Enabling python 3.13 in the pyproject.toml and also ensure that the test run over all supported python versions.

Currently fails to install as argilla does not support python 3.13, but i will do so soon.

Before Merging

  • Review the code changes
    • Unused print / comments / TODOs
    • Missing docstrings for functions that should have them
    • Consistent variable names
    • ...
  • Update changelog.md if necessary
  • Commit messages should contain a semantic label and the ticket number
    • Consider squashing if this is not the case

@JanEricNitschkeAA JanEricNitschkeAA changed the title Chore: add python 313 Draft: Chore: add python 313 Nov 26, 2024
@JanEricNitschkeAA JanEricNitschkeAA marked this pull request as draft November 26, 2024 14:49
@JanEricNitschkeAA JanEricNitschkeAA changed the title Draft: Chore: add python 313 Chore: add python 313 Nov 26, 2024
@JanEricNitschkeAA
Copy link
Contributor Author

Now there was an issue with grpcio-tools which does support 3.13 but has no prebuilt wheels for the resolved version (1.62.3).

I bumped the opentelemtry versions to get to one that does (1.68).

However now lingua-language-detector causes problems because it does not have 3.13 wheels.

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.

1 participant