<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"><head><meta http-equiv=Content-Type content="text/html; charset=utf-8"><meta name=Generator content="Microsoft Word 15 (filtered medium)"><style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:#954F72;
text-decoration:underline;}
.MsoChpDefault
{mso-style-type:export-only;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style></head><body lang=EN-US link=blue vlink="#954F72"><div class=WordSection1><p class=MsoNormal>Here’s the latest patch. The result code output from X3DJSAIL still doesn’t work, but changes should be able to be applied without breaking anything. The usage documentation is unchanged.</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal><a href="https://github.com/coderextreme/JavaSceneAuthoringInterfaceLibrary/blob/master/CreateX3DJSAI.patch">https://github.com/coderextreme/JavaSceneAuthoringInterfaceLibrary/blob/master/CreateX3DJSAI.patch</a></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Sorry, I don’t really know what classes the code goes in. If I get the time, I will do some searching.</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>I believe the rest of the changes besides X3DObject.java are in CommandLine.java, but I don’t really know.</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>John</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Sent from <a href="https://go.microsoft.com/fwlink/?LinkId=550986">Mail</a> for Windows 10</p><p class=MsoNormal><o:p> </o:p></p><div style='mso-element:para-border-div;border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in'><p class=MsoNormal style='border:none;padding:0in'><b>From: </b><a href="mailto:brutzman@nps.edu">Don Brutzman</a><br><b>Sent: </b>Sunday, October 1, 2017 10:44 PM<br><b>To: </b><a href="mailto:yottzumm@gmail.com">John Carlson</a><br><b>Cc: </b><a href="mailto:x3d-public@web3d.org">x3d-public@web3d.org</a><br><b>Subject: </b>Re: Patch for X3DJSAIL</p></div><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>On 9/28/2017 8:11 PM, John Carlson wrote:</p><p class=MsoNormal>> Patch for CreateX3dSceneAccessInterfaceJava.xslt</p><p class=MsoNormal>> </p><p class=MsoNormal>> Important things:</p><p class=MsoNormal>> </p><p class=MsoNormal>> 1. <String>’s on line 340</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>OK have modified it to match your preferred form. Not sure why though, prior </p><p class=MsoNormal>construct didn't have any warnings. Plus similar constructs exist for other </p><p class=MsoNormal>types. Is this a pyjnius transcoding issue?</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>340c340</p><p class=MsoNormal>< <xsl:text disable-output-escaping="yes">new </p><p class=MsoNormal>ArrayList<>(Arrays.asList(</xsl:text><!-- ArrayList<> --></p><p class=MsoNormal>---</p><p class=MsoNormal> > <xsl:text disable-output-escaping="yes">new </p><p class=MsoNormal>ArrayList<String>(Arrays.<String>asList(</xsl:text><!-- ArrayList<> --></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>> 2. Imports</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Please advise what this is for - perhaps add a more verbose comment as well?</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Also please reconfirm class of interest, there have been some edits so I didn't </p><p class=MsoNormal>want to put it in the wrong place.</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>762a763,766</p><p class=MsoNormal> > // Script operations</p><p class=MsoNormal> > import javax.script.ScriptEngineManager;</p><p class=MsoNormal> > import javax.script.ScriptEngine;</p><p class=MsoNormal> > import javax.script.ScriptException;</p><p class=MsoNormal>2958c2962</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>> 3. The rest would be nice to have, because I have to keep updating the</p><p class=MsoNormal>> stylesheet, but I don’t have a “working” version yet.</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Am reluctant to change directory structure, the entire build process depends on </p><p class=MsoNormal>it and not sure why you need it:</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>41c41</p><p class=MsoNormal>< <xsl:variable name="saiPackageDirectorySource"> </p><p class=MsoNormal><xsl:text>src/org/web3d/x3d/sai</xsl:text></xsl:variable></p><p class=MsoNormal>---</p><p class=MsoNormal> > <xsl:variable name="saiPackageDirectorySource"> </p><p class=MsoNormal><xsl:text>src/main/java/org/web3d/x3d/sai</xsl:text></xsl:variable></p><p class=MsoNormal>46c46</p><p class=MsoNormal>< <xsl:variable name="concretePackageDirectorySource"> </p><p class=MsoNormal><xsl:text>src/org/web3d/x3d/jsail</xsl:text></xsl:variable></p><p class=MsoNormal>---</p><p class=MsoNormal> > <xsl:variable name="concretePackageDirectorySource"> </p><p class=MsoNormal><xsl:text>src/main/java/org/web3d/x3d/jsail</xsl:text></xsl:variable></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>The following change in X3DObject was superfluous and not needed:</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal> FILE_EXTENSION_JAVASCRIPT</p><p class=MsoNormal>to</p><p class=MsoNormal> X3DObject.FILE_EXTENSION_JAVASCRIPT</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>> It does create an appropriate JavaScript file, butX3DJSAIL doesn’t work with this command:</p><p class=MsoNormal>> </p><p class=MsoNormal>> $ java -cp </p><p class=MsoNormal>> "target/X3DJSONLD-1.0-SNAPSHOT.jar;../pythonSAI/X3DJSAIL.3.3.full.jar;saxon9he.jar" </p><p class=MsoNormal>> net.x3djsonld.data.abox js growl.js</p><p class=MsoNormal>> </p><p class=MsoNormal>> [ growl.sh in X3DJSONLD ]</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Last nights changes should fix any need to have saxon9he.jar in classpath. Use </p><p class=MsoNormal>X3DJSAIL.3.3.full.jar to test that out. Don't miss the ant build changes in </p><p class=MsoNormal>parent stylesheets/ directory, they are significant. Online at</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>https://sourceforge.net/p/x3d/code/HEAD/tree/www.web3d.org/x3d/stylesheets/CreateX3dSceneAccessInterfaceJava.xslt</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Updated today: added -validate switch and various improvements for CommandLine </p><p class=MsoNormal>and X3DLoaderObject. Also noted some missing min/max inclusive/inclusive values </p><p class=MsoNormal>in X3D Unified Object Model; these are now listed in X3D XML Schemas (3.2 3.3 </p><p class=MsoNormal>4.0), the object model file, and corresponding X3DJSAIL validation methods.</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>I tried to apply your patch using Netbeans > Tools > Apply Diff Patch but it </p><p class=MsoNormal>didn't work, probably because I had made too many changes.</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Am hoping you can redo the patch or (likely better) just send me the plain code </p><p class=MsoNormal>blocks and indicate class name(s) where to add them.</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>> Attached,</p><p class=MsoNormal>> </p><p class=MsoNormal>> John</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>thanks, let's keep iterating please. 8)</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>p.s. just released: Java 9. haven't tried it yet.</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal> https://docs.oracle.com/javase/9/whatsnew/toc.htm</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>all the best, Don</p><p class=MsoNormal>-- </p><p class=MsoNormal>Don Brutzman Naval Postgraduate School, Code USW/Br brutzman@nps.edu</p><p class=MsoNormal>Watkins 270, MOVES Institute, Monterey CA 93943-5000 USA +1.831.656.2149</p><p class=MsoNormal>X3D graphics, virtual worlds, navy robotics http://faculty.nps.edu/brutzman</p><p class=MsoNormal><o:p> </o:p></p></div></body></html>