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

Update pip.md #2251

Closed
wants to merge 1 commit into from
Closed

Conversation

ForeverZyh
Copy link
Contributor

Swap the order of $CUDNN_PATH/lib and CONDA_PREFIX/lib.

I was hitting an error, Loaded runtime CuD NN library: 8.1.0 but source was compiled with: 8.6.0.
I find out that my CONDA_PREFIX/lib contains libcudnn8.1.0 so TF will first use this version instead of the libcudnn8.6.0 installed in the CONDA_PREFIX/lib.

Swap the order of $CUDNN_PATH/lib and CONDA_PREFIX/lib
@8bitmp3
Copy link
Contributor

8bitmp3 commented Aug 15, 2023

@markmcd

@vijayasarathib
Copy link
Contributor

Looks good to me! The change makes sense given the issue described with the CuDNN library versions. I approve this change.

Copy link
Member

@markmcd markmcd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure how prevalent this issue is, if you have any more context on how you got into that state it'd be helpful.

That said, this is harmless, and I am generally in favour of ordering $PATH vars by specificity, which this does.

@github-actions github-actions bot added the lgtm Community-added approval label Aug 21, 2023
@markmcd markmcd added the ready to pull Start merge process label Aug 21, 2023
@ForeverZyh ForeverZyh closed this Aug 21, 2023
copybara-service bot pushed a commit that referenced this pull request Aug 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Community-added approval ready to pull Start merge process
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants