[x3d-public] Errors in Geometry2dComponentPrototypes.wrl at web3d.org

[Don Brutzman]

Thank you Sergey, спасибо for reporting this simple prolem.  The points -> point problem is corrected just as you suggested, and now matches the specification.

	http://www.web3d.org/documents/specifications/19775-1/V3.3/Part01/versionContent.html#Polypoint2D
	http://www.web3d.org/x3d/content/X3dTooltips.html#Polypoint2D

The metadata node is a little more interesting.  The current design has them in the prototype declaration as SFNode field declarations, but not connected anywhere inside the prototype body since they are not valid VRML97.  I think the original motivation was to not throw them away, since some X3D players which retained backwards compatibility might use them.

If you think these are harmful to include since they inject Metadata nodes into an unsuspecting VRML browser, that seems pretty reasonable.  If so, then I can omit them as you suggest, or possibly turn those statements into comments when translating from .x3d to .wrl file encoding.  Please advise.

Meanwhile the "point" changes are checked in and upload online.  Test reports:

Pass: InstantReality, freewrl (but then crashes after a bit), H3DViewer, X3DOM, Cobweb
Fail: BS Contact, Octaga, view3dscene, Xj3D

Updated scenes and updated X3dToVrml97.xslt stylesheet all checked into version control.  Available via

	http://www.web3d.org/x3d/content/examples/Basic/development#Geometry2dComponentExamples

The Cobweb result is new, and looks good.  Attached. 8)


On 6/28/2016 10:47 AM, Bederov, Sergey wrote:
> Hello all,
>
>
>
> Recently I have seen a VRML file converted from X3D by some converter. The file is using some non-standard VRML nodes and has EXTERNPROTO definitions pointing to:
>
>
>
> http://www.web3d.org/x3d/content/examples/Basic/development/Geometry2dComponentPrototypes.wrl
>
>
>
> I want to say that there are errors in that file:
>
>
>
> Line 45:
>
>           metadata IS metadata
>
>
>
> This field is contained in a “Group” node, but in VRML this node doesn’t have a “metadata” field. Here, this link can safely be removed because this field is not used anywhere.
>
>
>
> Line 497:
>
>           field        MFVec2f         points IS points
>
>
>
> The “points” field is not declared in the PROTO declaration, but the “point” field is declared instead. Replacing “points” to “point” seems to make it work.
>
>
>
> To whom it may concern, please fix these errors. Currently, that file is an invalid VRML file, and a well-behaving VRML browser doesn’t load it and therefore can’t use the PROTO declarations contained therein, even if they don’t have errors.
>
>
>
>
>
> Thank you,
>
> Sergey Bederov
>
>
>
>
>
> _______________________________________________
> x3d-public mailing list
> x3d-public at web3d.org
> http://web3d.org/mailman/listinfo/x3d-public_web3d.org
>


all the best, Don
-- 
Don Brutzman  Naval Postgraduate School, Code USW/Br       brutzman at nps.edu
Watkins 270,  MOVES Institute, Monterey CA 93943-5000 USA   +1.831.656.2149
X3D graphics, virtual worlds, navy robotics http://faculty.nps.edu/brutzman
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Geometry2dComponentExamplesCobweb.png
Type: image/png
Size: 18830 bytes
Desc: not available
URL: <http://web3d.org/pipermail/x3d-public_web3d.org/attachments/20160630/8fa6e3f0/attachment-0001.png>