<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">---------- Forwarded message ---------<br>From: <strong class="gmail_sendername" dir="auto">John Carlson</strong> <span dir="auto"><<a href="mailto:yottzumm@gmail.com">yottzumm@gmail.com</a>></span><br>Date: Sun, Nov 22, 2020 at 8:10 PM<br>Subject: Re: x3dom bug related to IS/children PROTOs?<br>To: Andreas Plesch <<a href="mailto:andreasplesch@gmail.com">andreasplesch@gmail.com</a>><br></div><br><br><div dir="ltr">I'll just pretend we zoomed back from 2020->2015 virtually (in the mind).<div><br><div>Can we agree, that in most cases (nodes), USE is the only field allowed when USE appears?</div><div><br></div><div>If not, I've got to change all my JSON schema generating code.</div><div><br></div><div>Maybe we'll decide on another JSON schema based on the XML schema, since XML Spy can now generate a JSON schema from an XML schema last I heard.</div><div><br></div><div>Let's just break all my coding for the last 5 years!  Whee!</div><div><br>I would need to know all the nodes/statements where additional fields are allowed when USE appears.  I think there's only two cases.</div><div><br></div><div>Or is this a thing in X3D4?</div><div><br></div><div>It seems like USE has two different semantics, whether you're using it on a node or not?  I know in one case, it's intended to add another parent to a node (not statement I guess), but perhaps that's a X3D3 thing?</div><div><br></div><div>John</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Nov 22, 2020 at 7:51 PM Andreas Plesch <<a href="mailto:andreasplesch@gmail.com" target="_blank">andreasplesch@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">I think the spec. does not have an opinion on this, see x3d-public and other players. Cheers, -Andreas</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Nov 22, 2020 at 8:47 PM John Carlson <<a href="mailto:yottzumm@gmail.com" target="_blank">yottzumm@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Note that X3DOM should now probably flag ProtoInstances with both name and USE with a warning that the code is now non-compliant or deprecated. I believe the spec says that USE and name together are only allowed in certain nodes (I think Metadata and perhaps one other comes to mind).<div><br></div><div>That's the best I have right now.  See previous discussion on mailing list regarding extra fields including name along with USE.</div><div><br></div><div>Also, if you use JSON schema, it should catch JSON examples as non-compliant if they contain both name and USE.  This works in JSON, because containerFields are not used in JSON.</div><div><br></div><div>And yes, this may have all changed under my nose!</div><div><br></div><div>John</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Nov 22, 2020 at 7:19 PM Andreas Plesch <<a href="mailto:andreasplesch@gmail.com" target="_blank">andreasplesch@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Yes, the Diamond files have the diamond shape in the ProtoDeclaration.<div><br></div><div>I added guards for nonimplemented but expected, standard fields (such as addChildren) to ProtoDeclaration.js, and allowed USE ProtoInstances by looking for DEFs in the dom (before parsing, at the initial step of translation to the concise x3dom syntax).</div><div><br></div><div>These changes should help with making these older HAnim protos work better.</div><div><br></div><div>-Andreas</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Nov 22, 2020 at 4:43 PM John Carlson <<a href="mailto:yottzumm@gmail.com" target="_blank">yottzumm@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><br></div><div dir="auto">I see one Shape (in a proto?), but I didn’t look for geometry.   I believe there is satisfactory appearance.   I believe I only checked one LOA4 file.   I am not sure about converted files.</div><div dir="auto"><br></div><div dir="auto">I will look into this in more detail later today.  I believe at least the Diamond files should have shapes.</div><div dir="auto"><br></div><div dir="auto">John</div><div dir="auto"><br><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr">On Sun, Nov 22, 2020 at 9:35 AM Andreas Plesch <<a href="mailto:andreasplesch@gmail.com" target="_blank">andreasplesch@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Yes this is Proto related, due to the "addChildren" and "removeChildren" fields not being available in x3dom for grouping nodes. And those are not available because x3dom cannot route node type fields.</div><div><br></div><div>Although the Protos define the addChildren field it is actually never used in the examples, so they are not actually necessary. You could just remove them.</div><div><br></div><div>The examples use an old Proto implementation of HAnim. The native implementation is also available, without Protos.</div><div><br></div><div>Perhaps it would make sense to add noop stubs for add/removeChildren fields in X3DGroupingNode.js, or add a guard for the Proto field type.</div><div><br></div><div>I notice that hanimloa4.html does not have any geometry to render, only the joint structure.</div></div>
</blockquote></div></div>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div>Andreas Plesch<br>Waltham, MA 02453</div></div></div>
</blockquote></div>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div>Andreas Plesch<br>Waltham, MA 02453</div></div></div>
</blockquote></div>
</div></div>