-
Notifications
You must be signed in to change notification settings - Fork 211
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
Apple Silicon Support #69
Comments
I don't have an M1 mac but would accept a PR that improved support for M1. cibuildwheel https://cibuildwheel.readthedocs.io/en/stable/#what-does-it-do says it supports apple silicon, so there's probably some way to make the github actions workflow build a wheel for that platform. |
@olmoulin Hi, and thanks for the solution you mentioned! Thank you :) |
Sure, install home-brew, it should be located in /opt/homebrew Hope it helps :) |
Thanks for the answer above! I found this worked for me without editing the CMakeLists.txt file. |
Following instructions from openai#69
If |
|
Ok, I manage to fix the errors I had by commenting out the following lines in chaser.cpp:296: And by replacing cpp:231:18 in starpilot.cpp with: |
@raymondchua this seems to be a breaking behavior of c compiler |
Problem: As mentioned in [this comment](openai#69 (comment)) of openai#69, a compilation crashes due to an error of unused variables on Apple silicon (with clang 14 and possibly 13+). This issue has been reported [elsewhere](ClickHouse/ClickHouse#27705). This seems to be caused by breaking behavior in clang 13 which escalates unused variable from a warning in prior versions to an error. The solution implemented here: I add an additional cflags `-Wno-unused-but-set-variable` for apple platforms. Tested on macOS 13.3.1 with Apple clang version 14.0.3 (clang-1403.0.22.14.1)
Thanks for following up, @falcondai. My clang version: Apple clang version 14.0.3 (clang-1403.0.22.14.1) |
hi, I was able to get progcen works on my macbook M1 by doing the following:
Install glfw with homebrew (and make sure the lib path is in DYLD_LIBRARY_PATH)
modify the march=native to mcpu=lightning in the cmake configuration file ...
It is quite an easy fix, so may be good if you want to implement it ...
The text was updated successfully, but these errors were encountered: