Skip to content

Update requirements.txt #21062

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Update requirements.txt #21062

wants to merge 4 commits into from

Conversation

sachinprasadhs
Copy link
Collaborator

Since Tensorflow and Tensorfow-cpu 2.19.0 is released, updating the version, also removed the version pin of JAX which was depending on the Tensorflow 2.19 release.

Since Tensorflow and Tensorfow-cpu 2.19.0 is released, updating the version, also removed the version pin of JAX which was depending on the Tensorflow 2.19 release.
@sachinprasadhs
Copy link
Collaborator Author

Not sure why the tests are failing with memory error!

@gbaned gbaned added this to PR Queue Mar 19, 2025
@github-project-automation github-project-automation bot moved this to Assigned Reviewer in PR Queue Mar 19, 2025
@gbaned gbaned requested a review from mattdangerw March 19, 2025 06:22
@fchollet
Copy link
Collaborator

Can you try only changing one of these things at a time to see which one is causing the test failure?

@fchollet
Copy link
Collaborator

It appears to be a cuDNN version issue?

E0000 00:00:1742492543.169566    2154 cuda_dnn.cc:8579] Unable to register cuDNN factory: Attempting to register factory for plugin cuDNN when one has already been registered

@sachinprasadhs
Copy link
Collaborator Author

It appears to be a cuDNN version issue?

E0000 00:00:1742492543.169566    2154 cuda_dnn.cc:8579] Unable to register cuDNN factory: Attempting to register factory for plugin cuDNN when one has already been registered

These warnings are there since long time with TensorFlow tensorflow/tensorflow#62075, I don't think that could be the issue.

@sampathweb
Copy link
Collaborator

@sachinprasadhs - I don't think we need both tensorflow-cpu and tensorflow in requirements.txt. May be they don't play well with each other anymore. Should we just keep cpu version for TF and Torch since that's what we have for jax?

@sachinprasadhs
Copy link
Collaborator Author

@sachinprasadhs - I don't think we need both tensorflow-cpu and tensorflow in requirements.txt. May be they don't play well with each other anymore. Should we just keep cpu version for TF and Torch since that's what we have for jax?

Made the changes, but any changes I make in this PR is resulting in memory error, not sure what is the reason.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Assigned Reviewer
Development

Successfully merging this pull request may close these issues.

4 participants