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

fatal error when running python setup.py install #7

Open
acunam opened this issue Jun 4, 2019 · 7 comments
Open

fatal error when running python setup.py install #7

acunam opened this issue Jun 4, 2019 · 7 comments

Comments

@acunam
Copy link

acunam commented Jun 4, 2019

when I run the python setup.py install command I get a fatal error with Geometry .cpp

path\Geometry.h(33): fatal error C1189: #error: OpenMP is required

error: command 'C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\BIN\x86_amd64\cl.exe' failed with exit status 2

any help with this issue is highly appreciated

@strongteck
Copy link

same issue here!

@zapaishchykova
Copy link

same, any solutions?

@mmolero
Copy link
Owner

mmolero commented Nov 7, 2020

I only tested using gcc compiler.

python setup.py build_ext --inplace --compiler=mingw32 -DMS_WIN64

@sonumathur
Copy link

@acunam have solved this issue.... if solved please provide solution

@acunam
Copy link
Author

acunam commented Apr 5, 2021

Unfortunately, I wasn't able to solve the issue. Finally, I successfully implemented the Poisson surface reconstruction that comes with Open3D. http://www.open3d.org/docs/latest/tutorial/Advanced/surface_reconstruction.html

@yosmo78
Copy link

yosmo78 commented Jun 9, 2021

when using the cl.exe compiler, what I did to make it work was in setup.py look at extra_compile_args = ["-w","-fopenmp"]

change -fopenmp to /openmp since microsoft doesn't support the -fopenmp flag, but instead uses /openmp for enabling openmp as per here

I also did it in the src folder setup.py too. you can also change it in the extra_link_args=["-fopenmp"] line as well for both files, but I am not sure it is necessary or not (I haven't played around with it enough to know, although the linker does give a warning when using it LINK : warning LNK4044: unrecognized option '/openmp'; ignored).

@cpatel245
Copy link

Hi @acunam
I am also facing same issue. do you have your solution on git repo? if yes, would you be able to share it?

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

No branches or pull requests

7 participants