loryhndol / LowpolyBuildings

Implementation of Low-poly mesh generation for building models
24 stars 4 forks source link

Huge amount of calculation & CGAL error #2

Closed HsiaoYeekwan closed 1 year ago

HsiaoYeekwan commented 1 year ago

Thanks for your reply~ When I run using resources\bunny10K.obj, there are 'Direction Set' contains so many vector that a huge amount of calculation.

(eg: The for loop will be executed 49924100* 49924100 times) image

And I also encountered a CGAL error when I delete some values in 'Direction Set': image

loryhndol commented 1 year ago

Indeed, this is a bug. The old version of code made mistakes when fitPlanesFromRegions, the link to code shows the correct sequence to execute. (outside the inner loop)