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

working with python 3.12 #2143

Closed
wants to merge 1 commit into from
Closed

working with python 3.12 #2143

wants to merge 1 commit into from

Conversation

johnnynunez
Copy link

Fix #2129 and #1846

@glenn-jocher
Copy link

@johnnynunez nice!

@johnnynunez johnnynunez mentioned this pull request Feb 14, 2024
@TobyRoseman
Copy link
Collaborator

This looks promising for fixing #1846.

However, additional changes will be necessary to support Python 3.12 (#2143). We'll need to update the versions of TensorFlow and PyTorch we use for testing. We'll need to use versions that support Python 3.12.

@johnnynunez
Copy link
Author

This looks promising for fixing #1846.

However, additional changes will be necessary to support Python 3.12 (#2143). We'll need to update the versions of TensorFlow and PyTorch we use for testing. We'll need to use versions that support Python 3.12.

tensorflow is only supporting on nightly version.
Pytorch is compatible

@johnnynunez
Copy link
Author

tensorflow

tensorflow is now compatible with python 3.12
Release candidate v2.16.0-rc0

@TobyRoseman
Copy link
Collaborator

@johnnynunez - yes, the first release candidate for TensorFlow 2.16.0 does have 3.12 wheels. I think we'll need to wait at least until the TensorFlow's official 2.16 release before we move forward with 3.12 support.

@johnnynunez
Copy link
Author

@johnnynunez - yes, the first release candidate for TensorFlow 2.16.0 does have 3.12 wheels. I think we'll need to wait at least until the TensorFlow's official 2.16 release before we move forward with 3.12 support.

I'm agree

@johnnynunez
Copy link
Author

johnnynunez commented Mar 12, 2024

@TobyRoseman @glenn-jocher tensorflow 2.16.1 is out compatible with Python 3.12.
Consider merge it...
Important thing is that TF-Estimators will not be available in TensorFlow 2.16 or after. It is included on tensorflow whl

@glenn-jocher
Copy link

@johnnynunez thanks! Unfortunately https://github.com/ultralytics/ultralytics is constrained by tflite-support to use tensorflow<2.13.1, so we've been pinned there for a long time now.

The tflite-support repo appears to be no longer maintained by Google, and we don't see any alternative for attaching metadata to our exported YOLOv8 TFLite models.

See tensorflow/tflite-support#954

@TobyRoseman
Copy link
Collaborator

TobyRoseman commented Mar 12, 2024

I don't think the except part of the change is necessary. We no longer support Python 3.5.

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.

Python 3.12 Support
3 participants