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

Fixes installation on MacOS on Apple Sillicon #77

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

wouterdebie
Copy link
Contributor

  • Bumps py-cpuinfo to version 9
  • Since there are no precompiled llama-cpp-python wheels available for MacOS on Apple Sillicon, this installs direclty from pypiy.org

Copy link
Owner

@gokayfem gokayfem left a comment

Choose a reason for hiding this comment

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

can you check the starting time for node, newer version of py-cpuinfo caused very long starting time issues. it should be around 0.4-0.6 seconds.

@wouterdebie
Copy link
Contributor Author

I'm a total noob when it comes to ComfyUI. What exactly do you want me to do? And how do I measure the startup time?

@gokayfem
Copy link
Owner

when you start comfyui it gives you loading time for each node in the command line next to nodes

@wouterdebie
Copy link
Contributor Author

Gotcha!

   0.3 seconds: /Users/wouter.de.bie/prjs/others/c2/ComfyUI/custom_nodes/ComfyUI_VLM_nodes

@gokayfem
Copy link
Owner

thanks, i will check for windows and linux and after that i will merge.

@wouterdebie
Copy link
Contributor Author

Great! If things are slow on Windows, I could make the version conditional.

@gokayfem
Copy link
Owner

gokayfem commented Apr 20, 2024

py-cpuinfo version 9 is slowing down loading time about 1.1 second on Windows.

- Bumps py-cpuinfo to version 9 for arm64, but keeps version 3 for x86_x64 due
  performance concerns.
- Since there are no precompiled llama-cpp-python wheels available for MacOS
  on Apple Sillicon, this installs direclty from pypiy.org.
@wouterdebie
Copy link
Contributor Author

wouterdebie commented Apr 20, 2024

I just pushed a new version that differentiates between arm64 and x86_64. Hope this works!

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.

None yet

2 participants