tudelft3d/AdTree

Error in density calculation.

harrydobbs opened this issue · 2 comments

harrydobbs commented

In skeleton.cpp on line 837:

dendiff = abs(currentDensity - ptDensity);

Shouldn't this be:

dendiff += abs(currentDensity - ptDensity);

As with the current implementation you are only look at the density of the point last iterated in the loop, rather than the neighborhood average?
Or is my understanding of the code wrong?

Cheers,
Harry

LiangliangNan commented

It is very likely to be an error. We will investigate it and get back to you later.
Thanks!

LiangliangNan commented

@harry1576 Indeed, it was an error. We have fixed it and the code has been updated.
Thank you again for reporting this error!

  • 👍1