-
Notifications
You must be signed in to change notification settings - Fork 136
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
chore: fixing OMP issues for macOS ARM and Python3.11. #870
base: main
Are you sure you want to change the base?
Conversation
bcm-at-zama
commented
Sep 11, 2024
- Looks like the OMP issues also happen on ARM
- And by the way, we didn't have the fix for Python3.11
@@ -10,7 +10,7 @@ MACHINE=$(uname -m) | |||
PYTHON_VERSION=$(python --version | cut -d' ' -f2 | cut -d'.' -f1,2) | |||
DO_REGENERATE=0 | |||
|
|||
if [ "$UNAME" == "Darwin" ] && [ "$MACHINE" != "arm64" ] |
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.
I don't know how it works for you, @fd0r @RomanBredehoft
Could you tell me?
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.
I'm getting an error when running make sybc_env
elif [ "$PYTHON_VERSION" == "3.11" ] | ||
then | ||
rm "${WHICH_VENV}"/lib/"${WHICH_PYTHON}"/./site-packages/xgboost/.dylibs/libomp.dylib | ||
ln -s "${WHICH_VENV}"/lib/"${WHICH_PYTHON}"/site-packages/concrete/.dylibs/libomp.dylib "${WHICH_VENV}"/lib/"${WHICH_PYTHON}"/./site-packages/xgboost/.dylibs/libomp.dylib |
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.
completely trivial to add now, thanks to the DO_REGENERATE=1
option!
we'll be able to add 3.12 easily
dac8529
to
19b30c8
Compare
echo "Please have a look to libraries libiomp5.dylib related to torch and then" | ||
echo "apply appropriate fix" | ||
exit 255 | ||
# The error is specific to python version |
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.
So finally I haven't changed the things for Intel, and I have created a new category for ARM, where all python are equivalent