-
Notifications
You must be signed in to change notification settings - Fork 47
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
Failed to install on MacOS #35
Comments
Hi Muhammad,
Thank you for reporting this, and I am sorry that it didn't work for you.
From the error message, it appears that the problem is that your compiler
does not support OpenMP, which is used in Deepwave. It seems to be possible
to install one that does:
https://stackoverflow.com/questions/60005176/how-to-deal-with-clang-error-unsupported-option-fopenmp-on-travis
. Are you able to do that, and, if so, does it solve the problem?
|
Hi Allen,
I have tried the solution you suggested. Unfortunately, it did not work. It
still gives the same message.
Thanks!
…On Tue, Sep 28, 2021 at 5:00 PM Alan Richardson ***@***.***> wrote:
Hi Muhammad,
Thank you for reporting this, and I am sorry that it didn't work for you.
From the error message, it appears that the problem is that your compiler
does not support OpenMP, which is used in Deepwave. It seems to be possible
to install one that does:
https://stackoverflow.com/questions/60005176/how-to-deal-with-clang-error-unsupported-option-fopenmp-on-travis
<https://urldefense.com/v3/__https://stackoverflow.com/questions/60005176/how-to-deal-with-clang-error-unsupported-option-fopenmp-on-travis__;!!Nmw4Hv0!nZOXo9OmLEK4-ICr6DCXPZGhPkvwZq2R0DfWO3vx3F-ZowUzrUDu2cRLKUL3968j1oVYSqZjHhIC$>
. Are you able to do that, and, if so, does it solve the problem?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<https://urldefense.com/v3/__https://github.com/ar4/deepwave/issues/35*issuecomment-929041136__;Iw!!Nmw4Hv0!nZOXo9OmLEK4-ICr6DCXPZGhPkvwZq2R0DfWO3vx3F-ZowUzrUDu2cRLKUL3968j1oVYSjbdGpRr$>,
or unsubscribe
<https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/ALH7GBE3BEPWDBFL6XGWGPTUEGG5NANCNFSM5E4RVIJQ__;!!Nmw4Hv0!nZOXo9OmLEK4-ICr6DCXPZGhPkvwZq2R0DfWO3vx3F-ZowUzrUDu2cRLKUL3968j1oVYSvqwgtXW$>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://urldefense.com/v3/__https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675__;!!Nmw4Hv0!nZOXo9OmLEK4-ICr6DCXPZGhPkvwZq2R0DfWO3vx3F-ZowUzrUDu2cRLKUL3968j1oVYSgXb7_P4$>
or Android
<https://urldefense.com/v3/__https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign*3Dnotification-email*26utm_medium*3Demail*26utm_source*3Dgithub__;JSUlJSU!!Nmw4Hv0!nZOXo9OmLEK4-ICr6DCXPZGhPkvwZq2R0DfWO3vx3F-ZowUzrUDu2cRLKUL3968j1oVYStkGGUlF$>.
--
*Muhammad Izzatullah*
King Abdullah University of Science and Technology (KAUST)
Division of Physical Sciences and Engineering
Bldg. 1, Desk #0202-WS023
Thuwal 23955-6900, Kingdom of Saudi Arabia
*Tel*.: +966 (56) 569 7388
*Email*: ***@***.***
***@***.***>
*Seismic Modeling & Inversion (SMI) Research Group*
https://smi.kaust.edu.sa/
--
This message and its contents, including attachments are intended solely
for the original recipient. If you are not the intended recipient or have
received this message in error, please notify me immediately and delete
this message from your computer system. Any unauthorized use or
distribution is prohibited. Please consider the environment before printing
this email.
|
Hi Muhammad,
My guess is that pip is not using the new compiler that you installed
with that brew command, but instead still using the old one that does
not include OpenMP support. The second part of that answer (export
CPP=/usr/local/opt/llvm/bin/clang) sets the environment variable CPP
that is often used to choose which compiler to use for C++ to point to
the new compiler, but for the Deepwave case (which uses PyTorch's
CPPExtension to compile the code), it seems like it should be CXX
instead, which might be why it didn't work.
But first, if you want to check that the new compiler works with
OpenMP, you could create an empty file test.cpp and then run c++ -c
-fopenmp test.cpp and see if it fails with a message saying that
-fopenmp is not supported. This used the old compiler and so this is
expected. You could then repeat this with
/usr/local/opt/llvm/bin/clang -c -fopenmp test.cpp to verify that the
new compiler does support OpenMP (it should not give an error
message).
If that works, then the next task is to make sure that pip uses the
right compiler. In the same terminal where you are going to run pip,
run export CXX=/usr/local/opt/llvm/bin/clang and then run pip install
deepwave and it should hopefully use the right compiler.
|
Hi Allen, I tried your second solution, and it worked! I was able to install deepwave. Thank you so much! |
Hi Muhammad,
That's great news. I hope that you find it useful.
|
Thanks a lot! Deepwave is really useful for my research. :)
…On Wed, Sep 29, 2021 at 1:00 AM Alan Richardson ***@***.***> wrote:
Hi Muhammad,
That's great news. I hope that you find it useful.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<https://urldefense.com/v3/__https://github.com/ar4/deepwave/issues/35*issuecomment-929495059__;Iw!!Nmw4Hv0!jk3Rcr46vvnpRRTFBAlYVsWBlX6zaUaWLfFrqBPyEvD0HtuSU2gfBGH0k6lf7hbG8UDhhL9HrxC8$>,
or unsubscribe
<https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/ALH7GBGYK5FVPHOAWVX2GVLUEH7DPANCNFSM5E4RVIJQ__;!!Nmw4Hv0!jk3Rcr46vvnpRRTFBAlYVsWBlX6zaUaWLfFrqBPyEvD0HtuSU2gfBGH0k6lf7hbG8UDhhKTK58s-$>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://urldefense.com/v3/__https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675__;!!Nmw4Hv0!jk3Rcr46vvnpRRTFBAlYVsWBlX6zaUaWLfFrqBPyEvD0HtuSU2gfBGH0k6lf7hbG8UDhhNXRoUDQ$>
or Android
<https://urldefense.com/v3/__https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign*3Dnotification-email*26utm_medium*3Demail*26utm_source*3Dgithub__;JSUlJSU!!Nmw4Hv0!jk3Rcr46vvnpRRTFBAlYVsWBlX6zaUaWLfFrqBPyEvD0HtuSU2gfBGH0k6lf7hbG8UDhhEkNtn3e$>.
--
*Muhammad Izzatullah*
King Abdullah University of Science and Technology (KAUST)
Division of Physical Sciences and Engineering
Bldg. 1, Desk #0202-WS023
Thuwal 23955-6900, Kingdom of Saudi Arabia
*Tel*.: +966 (56) 569 7388
*Email*: ***@***.***
***@***.***>
*Seismic Modeling & Inversion (SMI) Research Group*
https://smi.kaust.edu.sa/
--
This message and its contents, including attachments are intended solely
for the original recipient. If you are not the intended recipient or have
received this message in error, please notify me immediately and delete
this message from your computer system. Any unauthorized use or
distribution is prohibited. Please consider the environment before printing
this email.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi,
I tried to install DeepWave on my Macbook. Unfortunately, the installation failed because it failed to build the wheel. Could you assist me with this? I attached the error below. Thank
error.docx
s!
The text was updated successfully, but these errors were encountered: