Skip to content

Mac builds#1037

Merged
LostRuins merged 10 commits intoLostRuins:concedo_experimentalfrom
henk717:concedo_experimental
Aug 4, 2024
Merged

Mac builds#1037
LostRuins merged 10 commits intoLostRuins:concedo_experimentalfrom
henk717:concedo_experimental

Conversation

@henk717
Copy link
Copy Markdown
Collaborator

@henk717 henk717 commented Aug 2, 2024

With some help of M.Holy I managed to find the issue with the metal pyinstaller, this CI should build and then test the binary for MacOS.

Mac still needs some overhauls for the UI since they can't input the gpulayers, and ideally we find a way to auto detect layers. But for now this at least allows binary builds.

@LostRuins LostRuins added the enhancement New feature or request label Aug 3, 2024
@LostRuins
Copy link
Copy Markdown
Owner

Very nice - what error happens if tkinter is loaded?

@henk717
Copy link
Copy Markdown
Collaborator Author

henk717 commented Aug 3, 2024

No error metal just needs its own backend screen with GPU layers exposed. No Blas is not a good fit for it.

@LostRuins
Copy link
Copy Markdown
Owner

Oh, but this currently has no metal support correct?

@henk717
Copy link
Copy Markdown
Collaborator Author

henk717 commented Aug 3, 2024

It does, it works as well as the non prebuilt does we solved the issues with metal.
Can be merged as is, but I recommend making a GUI for Metal prior to releasing binaries.

@LostRuins LostRuins merged commit 52cc0da into LostRuins:concedo_experimental Aug 4, 2024
LostRuins pushed a commit that referenced this pull request Aug 6, 2024
* OSX attempt 1

* OSX Pyinstaller

* Update kcpp-build-release-osx.yaml

* Update kcpp-build-release-osx.yaml

* Update kcpp-build-release-osx.yaml

* Add .metal file

* Update kcpp-build-release-osx.yaml

* Polish Mac

(cherry picked from commit 52cc0da)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants