OpenDroneMap / ODM

A command line toolkit to generate maps, point clouds, 3D models and DEMs from drone, balloon or kite images. 📷
https://opendronemap.org
GNU Affero General Public License v3.0
4.91k stars 1.11k forks source link

Fix PoissonRecon failed with n threads log message #1756

Closed andrewharvey closed 7 months ago

andrewharvey commented 7 months ago

The message was reporting failure with n threads and retrying with n // 2, however a few lines up threads was already set to n // 2 representing the next thread count to try.

You'll see currently the log finishing with

[WARNING] PoissonRecon failed with 1 threads, let's retry with 0...

which demonstrates the issues.

pierotofy commented 7 months ago

Nice catch @andrewharvey ! Thanks for the patch. 🙏