Faces with number of vertices greater or equal than 5 don't display properly about hangar HOT 3 CLOSED

Could you provided the file that you are using for testing?

From the library documentation: "Degenerated polygons are discarded. That is, polygons with less than three vertices, polygons with collinear coordinates, polygons with vertices not contained in an only one plane, polygons crossing edges, and so on."

Only the first three vertices are taken for normal computation. That's the expected behaviour.

And about the triangulator.js fix. I don't see it really necessary, because "u" takes the previous value of "v" on each iteration, and that value seems to be always fine.

from hangar.

Hi Juan,

Thank you for your response,
We had an issue with this model :
http://scenemodels.flightgear.org/app.php?c=Models&a=view&id=5186 whose
a great number of faces were invisible using Hangar, but visible in
FlightGear itself. Some other models also had this issue but not as much
as this one.
You can download it from "Download model" link.

About triangulator.js, it is possible to have "v = len -1", but in the
next loop, given that "u= v" and that "len" is drecreased by 1, "u" can
equal "len". At least, making sure that "u < len" fix the bug.

Of course, feel free to accept this bug fix or not.

Best regards,
Julien Nguyen

Le 31/01/2016 12:58, Juan Mellado a écrit :

Could you provided the file that you are using for testing?

From the library documentation: "Degenerated polygons are discarded.
That is, polygons with less than three vertices, polygons with
collinear coordinates, polygons with vertices not contained in an only
one plane, polygons crossing edges, and so on."

Only the first three vertices are taken for normal computation. That's
the expected behaviour.

And about the triangulator.js fix. I don't see it really necessary,
because "u" takes the previous value of "v" on each iteration, and
that value seems to be always fine.

—
Reply to this email directly or view it on GitHub
#6 (comment).

from hangar.

Ok, I'll try to fix the code along this week.

Thanks

from hangar.