search

mmolero / pypoisson

Poisson Surface Reconstruction Python Binding
MIT License
242 stars 30 forks source link

fatal error when running python setup.py install #7

Open acunam opened 5 years ago

acunam commented 5 years ago

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 commented 4 years ago

same issue here!

zapaishchykova commented 3 years ago

same, any solutions?

mmolero commented 3 years ago

I only tested using gcc compiler.

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

sonumathur commented 3 years ago

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

acunam commented 3 years ago

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 commented 3 years ago

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 commented 2 years ago

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?