Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Update requirements.txt#21062

Open
sachinprasadhs wants to merge4 commits intomaster
base:master
Choose a base branch
Loading
fromsachinprasadhs-patch-1
Open

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
CollaboratorAuthor

Not sure why the tests are failing with memory error!

@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
CollaboratorAuthor

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 TensorFlowtensorflow/tensorflow#62075, I don't think that could be the issue.

@sampathweb
Copy link
Collaborator

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

@sachinprasadhs
Copy link
CollaboratorAuthor

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

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 freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@mattdangerwmattdangerwAwaiting requested review from mattdangerw

Assignees

@gbanedgbaned

Projects
Status: Assigned Reviewer
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@sachinprasadhs@fchollet@sampathweb@gbaned

[8]ページ先頭

©2009-2025 Movatter.jp