nmellado / Super4PCS

Efficient Global Point-cloud registration
http://nmellado.github.io/Super4PCS/
Other
455 stars 215 forks source link

Bug during normalization #55

Closed pookiefoof closed 6 years ago

pookiefoof commented 6 years ago

Hi there,

I happened to find that in shared4pcs.h (line 89-91):

inline void normalize() {
    pos_.normalize();
}

I believe 3D positions should not be normalized like this, and this issue would lead to malfunctions when the input contains both positions and normals.

nmellado commented 6 years ago

Thanks for the report.