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

Bump tensorflow version #110

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Bump tensorflow version #110

wants to merge 1 commit into from

Conversation

stes
Copy link
Collaborator

@stes stes commented Feb 3, 2024

See #103. Cannot foresee all possible side implications (windows eg), but this would be a useful change, as google colab now ships tf 2.15, and installing dlc live first re-installs TF on colab:

  Attempting uninstall: tensorflow
    Found existing installation: tensorflow 2.15.0
    Uninstalling tensorflow-2.15.0:
      Successfully uninstalled tensorflow-2.15.0

Would it maybe be possible to remove the restriction fully?

@stes
Copy link
Collaborator Author

stes commented Feb 3, 2024

Seems like this URL needs to be fixed first:

image

@sneakers-the-rat
Copy link
Collaborator

looks like previously there was reason to remove cap anyway :): #103 (comment)

@MMathisLab
Copy link
Member

MMathisLab commented Feb 4, 2024

2.15 isn't compatible with dlc unfortunately... and now one needs to downgrade Cuda on Colab too (see open issues on main repo/PR from @n-poulsen. DeepLabCut/DeepLabCut@258b19b

Plan is to roll out dlc3 this month to avoid all of this 🙌

@AlexEMG
Copy link
Member

AlexEMG commented Feb 4, 2024

Yes, I think should open an issue clearly indicating at what TF/CUDA we stop here. And that new users should switch to: https://github.com/DeepLabCut/DeepLabCut-live-pytorch [WIP] 🚧

@sneakers-the-rat
Copy link
Collaborator

aha, i see last supported version of tf is 2.10: DeepLabCut/DeepLabCut#2509 (comment)

Yes, I think should open an issue clearly indicating at what TF/CUDA we stop here. And that new users should switch to: https://github.com/DeepLabCut/DeepLabCut-live-pytorch [WIP] 🚧

is it time to set a deprecation warning and get ready to say farewell to this olde package then? i can prepare the horn section and champagne for the ceremony

@MMathisLab
Copy link
Member

Yes! Actually we should set this warning when loaded; support ends at TF2.10 (for windows); and DLC is fully moving to pytorch 🤩🍾

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.

4 participants