Skip to content

Conversation

@henk717
Copy link
Collaborator

@henk717 henk717 commented Dec 13, 2023

No description provided.

root and others added 13 commits December 9, 2023 19:58
I know its technically amd64, but I don't want to cause confusion among nvidia users.
Unsure how this will behave in the pyinstaller build, but pocl ended up CPU only. With a bit of luck the pyinstaller uses the one from the actual system if compiled in a system without opencl, while conda now includes it for that specific system.
Missing this causes compile failures on some system's
Ideally we find a better solution, but conda forces ICD and needs this for the successful compile. However, pyinstaller then embeds the ICD causing it to be limited to the system it was compiled for. By temporarily removing the ICD pyinstaller can't find it and everything remains functional. Ideally we do this on a pyinstaller level, but I could not find any good options to do so yet.
@henk717 henk717 changed the title Fix & Nocuda Koboldcpp.sh Fix & Nocuda Dec 13, 2023
Copy link
Owner

@LostRuins LostRuins left a comment

Choose a reason for hiding this comment

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

Merging without testing

@LostRuins LostRuins merged commit 3067547 into LostRuins:concedo Dec 13, 2023
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.

2 participants