-
-
Notifications
You must be signed in to change notification settings - Fork 10
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
Add Windows + CUDA #6
Conversation
…nda-forge-pinning 2023.12.05.01.10.54
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
…nda-forge-pinning 2023.12.05.01.10.54
…nda-forge-pinning 2023.12.05.03.16.13
…nda-forge-pinning 2023.12.05.03.16.13
…nda-forge-pinning 2023.12.05.03.16.13
@conda-forge-admin, please rerender |
Hi! This is the friendly automated conda-forge-webservice. I tried to rerender for you, but it looks like there was nothing to do. This message was generated by GitHub actions workflow run https://github.com/conda-forge/llama.cpp-feedstock/actions/runs/7097997778. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Document that cuda-cudart_win-64 package is not populating automatically
- cuda-cudart {{ cuda_compiler_version }} # [(cuda_compiler_version or "").startswith("12")] | ||
- cuda-cudart_{{ target_platform }} {{ cuda_compiler_version }} # [(cuda_compiler_version or "").startswith("12") and win] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Something Is wrong with the Windows dependencies. Unless I add the line above I get the following error on Windows
ERROR (llama.cpp,bin/main.exe): Needed DSO Library/bin/cudart64_12.dll found in ['cuda-cudart_win-64']
ERROR (llama.cpp,bin/main.exe): .. but ['cuda-cudart_win-64'] not in reqs/run, (i.e. it is overlinking) (likely) or a missing dependency (less likely)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah this is due to upstream issue ( conda-forge/cuda-cudart-feedstock#13 (comment) )
We just merged a fix ( conda-forge/cuda-cudart-feedstock#14 ). So this should be resolved with the new packages
Think we can drop this then
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Raised issue ( #16 ) to track
Hi! This is the friendly conda-forge automerge bot! I considered the following status checks when analyzing this PR:
Thus the PR was passing and merged! Have a great day! |
Checklist
0
(if the version changed)conda-smithy
(Use the phrase@conda-forge-admin, please rerender
in a comment in this PR for automated rerendering)