<div dir="auto">Ok.   So the MF fields with type MFBool and MFFloat (as originally stated) may contain commas separating SF fields in the XML encoding?</div><div dir="auto"><br></div><div dir="auto">I think the  type SFVec3f for fieldOfView may be a misunderstanding?  We would need to tell everyone who has been using the field as an MFFloat like Holger?</div><div dir="auto"><br></div><div dir="auto">I will try to dig into tooltips, but I’m on my phone.</div><div dir="auto"><br></div><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jul 4, 2022 at 11:05 AM Brutzman, Donald (Don) (CIV) <<a href="mailto:brutzman@nps.edu">brutzman@nps.edu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)"><div lang="EN-US" link="blue" vlink="purple" style="word-wrap:break-word"><div class="m_-6916018500111536097WordSection1"><p class="MsoNormal">Hi John.<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><ol style="margin-top:0in" start="1" type="1"><li class="m_-6916018500111536097MsoListParagraph">Commas are treated as whitespace in ClassicVRML encoding.<u></u><u></u></li><li class="m_-6916018500111536097MsoListParagraph">Commas are not allowed within SF field values in XML encoding.  They are allowed between SF field values in MF field lists.<u></u><u></u></li><li class="m_-6916018500111536097MsoListParagraph">For really long numeric value arrays, this is a helpful restriction that helps localize errors and support tuple counting by authors<u></u><u></u></li><li class="m_-6916018500111536097MsoListParagraph">Validation tools (included X3D regexes) diagnose this.<u></u><u></u></li><li class="m_-6916018500111536097MsoListParagraph">This is documented in X3D Tooltips, which are based on XML encoding.  Example excerpts follow.<u></u><u></u></li></ol><p class="MsoNormal"><u></u> <u></u></p><ul style="margin-top:0in" type="disc"><li class="m_-6916018500111536097MsoListParagraph" style="margin-left:0in">X3D Tooltips: Field Types Table<u></u><u></u></li><li class="m_-6916018500111536097MsoListParagraph" style="margin-left:0in"><a href="https://www.web3d.org/x3d/tooltips/X3dTooltips.html#FieldTypesTable" target="_blank">https://www.web3d.org/x3d/tooltips/X3dTooltips.html#FieldTypesTable</a><u></u><u></u></li><li class="m_-6916018500111536097MsoListParagraph" style="margin-left:0in">SFColor, “<b><span style="color:rgb(238,85,0)">Warning:</span></b> comma characters within attribute values do not pass strict validation, and comma separators are not included in canonical form.”<u></u><u></u></li><li class="m_-6916018500111536097MsoListParagraph" style="margin-left:0in">MFColor, “<b><span style="color:rgb(238,85,0)">Warning:</span></b> comma characters within contained singleton 3-tuple SFColor values do not pass strict validation. <br><b><span style="color:rgb(68,119,119)">Hint:</span></b> comma characters are allowed as whitespace between individual SFColor values in the list.”<u></u><u></u></li></ul><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">So once again we see that being careful and strict about X3D XML validation is very helpful when converting to other encodings.<u></u><u></u></p></div></div><div lang="EN-US" link="blue" vlink="purple" style="word-wrap:break-word"><div class="m_-6916018500111536097WordSection1"><div><p class="MsoNormal"><span style="font-size:10pt;font-family:"Courier New""><u style="font-family:"Courier New""></u> <u style="font-family:"Courier New""></u></span></p><p class="MsoNormal"><span style="font-size:10pt;font-family:"Courier New"">all the best, Don<u style="font-family:"Courier New""></u><u style="font-family:"Courier New""></u></span></p><p class="MsoNormal"><span style="font-size:10pt;font-family:"Courier New"">-- <u style="font-family:"Courier New""></u><u style="font-family:"Courier New""></u></span></p><p class="MsoNormal"><span style="font-size:10pt;font-family:"Courier New"">Don Brutzman  Naval Postgraduate School, Code USW/Br        <a href="mailto:brutzman@nps.edu" target="_blank" style="font-family:"Courier New"">brutzman@nps.edu</a><u style="font-family:"Courier New""></u><u style="font-family:"Courier New""></u></span></p><p class="MsoNormal"><span style="font-size:10pt;font-family:"Courier New"">Watkins 270,  MOVES Institute, Monterey CA 93943-5000 USA    +1.831.656.2149<u style="font-family:"Courier New""></u><u style="font-family:"Courier New""></u></span></p><p class="MsoNormal"><span style="font-size:10pt;font-family:"Courier New"">X3D graphics, virtual worlds, Navy robotics https://</span> <span style="font-size:10pt;font-family:"Courier New""><a href="http://faculty.nps.edu/brutzman" target="_blank" style="font-family:"Courier New"">faculty.nps.edu/brutzman</a><u style="font-family:"Courier New""></u><u style="font-family:"Courier New""></u></span></p></div><p class="MsoNormal"><u></u> <u></u></p><div><div style="border-style:solid none none;border-top-width:1pt;padding:3pt 0in 0in;border-top-color:rgb(225,225,225)"><p class="MsoNormal"><b>From:</b> John Carlson <<a href="mailto:yottzumm@gmail.com" target="_blank">yottzumm@gmail.com</a>> <br><b>Sent:</b> Sunday, July 3, 2022 11:25 PM<br><b>To:</b> Brutzman, Donald (Don) (CIV) <<a href="mailto:brutzman@nps.edu" target="_blank">brutzman@nps.edu</a>><br><b>Cc:</b> X3D Graphics public mailing list <<a href="mailto:x3d-public@web3d.org" target="_blank">x3d-public@web3d.org</a>>; <a href="mailto:holger.seelig@gmail.com" target="_blank">holger.seelig@gmail.com</a><br><b>Subject:</b> Re: [x3d-public] X3dToJson.xslt, X3dToPython.xslt, OrthoViewpoint fieldOfView<u></u><u></u></p></div></div><p class="MsoNormal"><u></u> <u></u></p><div><div><p class="MsoNormal">My recall isn’t that good, and I can’t recall if /,/ — comma, is treated as white space in some field values and not in others.   I guess this is yet to be specified for new architecture documents—it should be discussed in encoding and binding standards.   I do think one can use commas between SF values in an MF value (not within) in XML.   I believe python requires commas within and between tuples and lists.<u></u><u></u></p></div><div><p class="MsoNormal"><u></u> <u></u></p></div><div><p class="MsoNormal">It maybe good to have test cases which aren’t possible with current validation.<u></u><u></u></p></div><div><p class="MsoNormal"><u></u> <u></u></p></div><div><p class="MsoNormal">I think this came up before in reference to HAnim Motion.<u></u><u></u></p></div><div><p class="MsoNormal"><u></u> <u></u></p></div><div><p class="MsoNormal">“Be liberal in what you accept and conservative in what you generate.”  — Jon Postel.<u></u><u></u></p></div><div><p class="MsoNormal"><u></u> <u></u></p></div><div><p class="MsoNormal">I totally understand the frustration of dealing with corners of XSLT.<u></u><u></u></p></div><div><p class="MsoNormal"><u></u> <u></u></p></div><div><p class="MsoNormal">I’m hoping something like Tidy can clear this up.<u></u><u></u></p></div><div><p class="MsoNormal"><u></u> <u></u></p></div><div><p class="MsoNormal">John<u></u><u></u></p></div><div><p class="MsoNormal"><u></u> <u></u></p><div><div><p class="MsoNormal">On Sun, Jul 3, 2022 at 11:24 PM Brutzman, Donald (Don) (CIV) <<a href="mailto:brutzman@nps.edu" target="_blank">brutzman@nps.edu</a>> wrote:<u></u><u></u></p></div><blockquote style="border-style:none none none solid;border-left-width:1pt;padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in;border-left-color:rgb(204,204,204)"><div><div><p class="MsoNormal">[Changed subject line for readability/searchability – this is about OrthoViewpoint fieldOfView]<u></u><u></u></p><p class="MsoNormal"> <u></u><u></u></p><ol start="1" type="1"><li class="m_-6916018500111536097m7639793683326362096msolistparagraph">Invalid source XML in your .x3d excerpt: simple types should not have commas within them.<u></u><u></u></li></ol><p class="MsoNormal"> <u></u><u></u></p><ul type="disc"><li class="m_-6916018500111536097m7639793683326362096msolistparagraph">OrthoViewpoint fieldOfView='-1.5, -1.5, 1.5, 1.5'<u></u><u></u></li></ul><p class="MsoNormal"> <u></u><u></u></p><ol start="2" type="1"><li class="m_-6916018500111536097m7639793683326362096msolistparagraph">OrthoViewpoint fieldOfView is defined as MFFloat in specification but ought to be SFVec4f, since it must always have four numbers defining ordered values (minimum_x, minimum_y, maximum_x, maximum_y).<u></u><u></u></li></ol><p class="MsoNormal"> <u></u><u></u></p><p class="MsoNormal">I am somehow locked out of mantis and so can’t verify what we previously said about this (if anything).  Seems like a simple specification erratum.<u></u><u></u></p><p class="MsoNormal"><br>Correlation confirmation: TextureProjectorParallel has fieldOfView SFVec4f.  Meanwhile Viewpoint GeoViewpoint and TextureProjector have SFFloat.<u></u><u></u></p><p class="MsoNormal"> <u></u><u></u></p><ol start="3" type="1"><li class="m_-6916018500111536097m7639793683326362096msolistparagraph">I think that the various fieldOfView types are handled in the stylesheet s correctly now, checked in.<u></u><u></u></li></ol><p class="MsoNormal"> <u></u><u></u></p><p class="MsoNormal">I have also applied a tentative fix to X3D DOCTYPE, Schema, Tooltips and X3DUOM, but will wait to deploy any updates (including X3DJSAIL, X3DPSAIL) until checking mantis to confirm status.<u></u><u></u></p><p class="MsoNormal"> <u></u><u></u></p><p class="MsoNormal">Thanks John for reporting this error, very helpful.<u></u><u></u></p><p class="MsoNormal"><span style="font-size:10pt;font-family:"Courier New""> </span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:10pt;font-family:"Courier New"">all the best, Don</span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:10pt;font-family:"Courier New"">-- </span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:10pt;font-family:"Courier New"">Don Brutzman  Naval Postgraduate School, Code USW/Br        <a href="mailto:brutzman@nps.edu" target="_blank" style="font-family:"Courier New"">brutzman@nps.edu</a></span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:10pt;font-family:"Courier New"">Watkins 270,  MOVES Institute, Monterey CA 93943-5000 USA    +1.831.656.2149</span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:10pt;font-family:"Courier New"">X3D graphics, virtual worlds, Navy robotics https://</span> <span style="font-size:10pt;font-family:"Courier New""><a href="http://faculty.nps.edu/brutzman" target="_blank" style="font-family:"Courier New"">faculty.nps.edu/brutzman</a></span><u></u><u></u></p><p class="MsoNormal"> <u></u><u></u></p><div style="border-style:solid none none;border-top-width:1pt;padding:3pt 0in 0in;border-top-color:rgb(225,225,225)"><p class="MsoNormal"><b>From:</b> x3d-public <<a href="mailto:x3d-public-bounces@web3d.org" target="_blank">x3d-public-bounces@web3d.org</a>> <b>On Behalf Of </b>John Carlson<br><b>Sent:</b> Sunday, July 3, 2022 12:10 AM<br><b>To:</b> X3D Graphics public mailing list <<a href="mailto:x3d-public@web3d.org" target="_blank">x3d-public@web3d.org</a>>; <a href="mailto:holger.seelig@gmail.com" target="_blank">holger.seelig@gmail.com</a><br><b>Subject:</b> [x3d-public] X3dToJson.xslt, X3dToPython.xslt, create3000/Library Test ParticleSystem possible issues. Though may not pass!<u></u><u></u></p></div><p class="MsoNormal"> <u></u><u></u></p><div><p class="MsoNormal">Don,<u></u><u></u></p><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal">Here are some issues with various X3dTo*.xslt:<u></u><u></u></p><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal">grep fieldOfView VolumeEmitter.py<br>VolumeEmitter.py:    OrthoViewpoint(description='OrthoViewpoint',fieldOfView=-1.5,-1.5,1.5,1.5),<u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal">$ grep fieldOfView VolumeEmitter*<br>VolumeEmitter.json:              "@fieldOfView":-1.5,-1.5,1.5,1.5<br>VolumeEmitter.x3d:        fieldOfView='-1.5, -1.5, 1.5, 1.5'/><u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal">In the JSON, the MFFloat is does not appear to be an array, with [] around the numbers.<u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal">In the python, there's no tuple or list markers denoting a tuple or list<u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal">Note in the XML, both the "," and the space is separating the floats in the MFFloat.  I haven't seen that the MFBools with ", " as separators is working, but perhaps the problem is more general.<u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal">Links to XML below.<u></u><u></u></p></div><div><p class="MsoNormal">====================================================<u></u><u></u></p></div><div><p class="MsoNormal">Holger,<u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal">I fully realize that these are test samples, and they may not pass, thus when I convert x3d  to python and run, these errors may appear (which may be totally valid).<u></u><u></u></p><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal">BoundedPhysicsModel.py<br>x3d.py package 4.0.64.1 loaded, have fun with X3D Graphics!<br>Traceback (most recent call last):<br>  File "C:\Users\john\X3DJSONLD\src\main\python\net\x3djsonld\Library\Tests\Components\ParticleSystems\BoundedPhysicsModel.py", line 82, in <module><br>    emitter=ConeEmitter(position=(0,-3,0),direction=(1,4,0),angle=0.4,speed=0.2,mass=3200),<br>  File "C:\Users\john\X3DJSONLD\venv\lib\site-packages\x3d\x3d.py", line 29419, in __init__<br>    self.direction = direction<br>  File "C:\Users\john\X3DJSONLD\venv\lib\site-packages\x3d\x3d.py", line 29450, in direction<br>    assertLessThanEquals('direction', direction, 1)<br>  File "C:\Users\john\X3DJSONLD\venv\lib\site-packages\x3d\x3d.py", line 1847, in assertLessThanEquals<br>    assert isLessThanEquals(value, maximum), fieldName + '=' + str(value) + ' fails assertLessThanEquals maximum=' + str(maximum)<br>AssertionError: direction=(1, 4, 0) fails assertLessThanEquals maximum=1<u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal">PointEmitter.py<br>x3d.py package 4.0.64.1 loaded, have fun with X3D Graphics!<br>Traceback (most recent call last):<br>  File "C:\Users\john\X3DJSONLD\src\main\python\net\x3djsonld\Library\Tests\Components\ParticleSystems\PointEmitter.py", line 92, in <module><br>    WindPhysicsModel(speed=0.8,gustiness=0.8,turbulence=2),<br>  File "C:\Users\john\X3DJSONLD\venv\lib\site-packages\x3d\x3d.py", line 99589, in __init__<br>    self.turbulence = turbulence<br>  File "C:\Users\john\X3DJSONLD\venv\lib\site-packages\x3d\x3d.py", line 99643, in turbulence<br>    assertZeroToOne('turbulence', turbulence)<br>  File "C:\Users\john\X3DJSONLD\venv\lib\site-packages\x3d\x3d.py", line 1818, in assertZeroToOne<br>    assert isZeroToOne(value), str(fieldName) + '=' + str(value) + ' fails assertZeroToOne requirements: value(s) must be in range [0..1]'<br>AssertionError: turbulence=2 fails assertZeroToOne requirements: value(s) must be in range [0..1]<u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal" style="margin-bottom:12pt">SurfaceEmitter.py<br>x3d.py package 4.0.64.1 loaded, have fun with X3D Graphics!<br>Traceback (most recent call last):<br>  File "C:\Users\john\X3DJSONLD\src\main\python\net\x3djsonld\Library\Tests\Components\ParticleSystems\SurfaceEmitter.py", line 50, in <module><br>    MetadataSet(DEF='children',name='children',reference='<a href="https://nam10.safelinks.protection.outlook.com/?url=http%3A%2F%2Ftitania.create3000.de%2F&data=05%7C01%7Cbrutzman%40nps.edu%7C40626d73a7eb413283ff08da5d85e75d%7C6d936231a51740ea9199f7578963378e%7C0%7C0%7C637925127310045334%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=uppYBcwQcvLhUfca%2F6AIIUfF2FssWTG%2BmmOSX6%2FuX1A%3D&reserved=0" target="_blank">http://titania.create3000.de</a>',<br>TypeError: MetadataSet.__init__() got an unexpected keyword argument 'appearance'<u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal">Original XML here:  <a href="https://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcreate3000%2FLibrary%2Ftree%2Fmain%2FTests%2FComponents%2FParticleSystems&data=05%7C01%7Cbrutzman%40nps.edu%7C40626d73a7eb413283ff08da5d85e75d%7C6d936231a51740ea9199f7578963378e%7C0%7C0%7C637925127310045334%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=z1GBaoqNl%2BDzwnMFSQTYJYenJEFXhgJAQe0r12hqeNA%3D&reserved=0" target="_blank">Library/Tests/Components/ParticleSystems at main · create3000/Library (github.com)</a><u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal">Attached generated python code.<u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div></div></div></div></div></div></blockquote></div></div></div></div></div></blockquote></div></div>