Missing normals error message on converted collada files about o3d-obsolete

[GoogleCodeExporter]

Original comment by [email protected] on 23 Jun 2009 at 2:36

[GoogleCodeExporter]

The error is a bug in the sketchup exporter. It's exporting the outlines as 
<lambert>
but <lambert> requires normals and sketchup is not supplying normals. Sketchup 
can
fix this by exporting <constant> for the lines. I tried this manually by 
editing the
.dae file inside the .kmz file and it worked (and I got a box with outlines)

Other solutions:

1) We can change the o3dConverter to do an "if the file is .kmz then attempt to
change any material used by a primitive that is marked to draw lines instead of
triangles as constant" and possibly a switch to turn that off if they ever 
decide to
fix sketchup. (If they did supply normals then lambert will work)

2) make o3djs.pack.prepare pack generate normals if they are missing. This is 
not a
very good solution because (a) it could be incredibly slow in JavaScript and (b)
there is not enough information to make normals (c) it's not what the file says 
to do.

3) make the shader builder generate constant even if the file calls for 
lambert. This
is also not a very good solution because (a) it would mean the user will not 
get what
they asked for. If they have 2 models, each using the same lambert material 
called
"wood" and one of those models is missing nornals then we'd have to generate 2
materials even though the file says there is only one material. The user then
searches for the material "wood" expecting only 1 and finds 2 making is app not 
work.
(b) it's not what the file says to do.

So, I guess, unless we can get new version of Sketchup asap I suggest solution 
#1. It
actually has the similar issues as solution #3 in that the file asked for 
lambert but
we are switching it to constant. If that material is used in 2 places and one of
those places has normals it will not be display with lambert since the material 
is
forced to constant but at least it keeps them out of the main code so they only
effect the sketchup path.

Original comment by [email protected] on 23 Jun 2009 at 3:05

[GoogleCodeExporter]

The issue is the lines.

Sketchup outputs lines for the outlines and marks them as using a lambert 
shader. A
lambert shader REQUIRES normals but there are no normals in the file. A fix is 
being
worked on.

Original comment by [email protected] on 24 Jun 2009 at 11:26

[GoogleCodeExporter]

Thank you much, gmangoog. Until the fix is done, I'll take a look in the
Sketchup-exported collada and see if there's enough information in there for me 
to
put together a perl script to massage the correct <lambert>s into <constant>s.

Thanks again!
--John

Original comment by [email protected] on 25 Jun 2009 at 1:43

[GoogleCodeExporter]

Original comment by [email protected] on 26 Jun 2009 at 7:02

• Changed state: Started

[GoogleCodeExporter]

This is fixed. Download the latest converter.

Original comment by [email protected] on 11 Jul 2009 at 6:16

• Changed state: Fixed