[brlcad-commits] SF.net SVN: brlcad:[55065] brlcad/trunk/doc/docbook/lessons/en/ mged15_creating_a_toy_truck.xml

[<br...@us...>]

Revision: 55065
          http://brlcad.svn.sourceforge.net/brlcad/?rev=55065&view=rev
Author:   brlcad
Date:     2013-04-05 14:51:48 +0000 (Fri, 05 Apr 2013)
Log Message:
-----------
update the inside command parameters to match the new rpp vertex behavior that was made to match the ordering of an arb8 in 7.14.6 (r33973).  the "right" parameters were ironically the ones already documented in a note when the lessons were written up in 2001 as having changed.  tracing back through the commit history, it looks like this however was entirely unintentional (i.e., it was a bug) that was documented here instead of getting fixed.

what had happened: circa 1985 (r373) 'in' command directly implemented rpp (matching arb8), circa 1987 (r1518) wdb rpp routine added (with "natural origin" but not matching arb8), circa 2000 (r18885) 'in' command updated to use libwdb (inadvertently changing the order), circa 2009 (r33973) wdb routine finally fixed to match arb8, and now the doc finally gets fixed to match.  so there ya have it, took three devs scratching heads for a day to figure this all out.

left a note on how to verify the right values ('l' command so as to not rely on graphical inspection) and the old command so the docs will still work with older releases.

Revision Links:
--------------
    http://brlcad.svn.sourceforge.net/brlcad/?rev=33973&view=rev
    http://brlcad.svn.sourceforge.net/brlcad/?rev=373&view=rev
    http://brlcad.svn.sourceforge.net/brlcad/?rev=1518&view=rev
    http://brlcad.svn.sourceforge.net/brlcad/?rev=18885&view=rev
    http://brlcad.svn.sourceforge.net/brlcad/?rev=33973&view=rev

Modified Paths:
--------------
    brlcad/trunk/doc/docbook/lessons/en/mged15_creating_a_toy_truck.xml

Modified: brlcad/trunk/doc/docbook/lessons/en/mged15_creating_a_toy_truck.xml
===================================================================
--- brlcad/trunk/doc/docbook/lessons/en/mged15_creating_a_toy_truck.xml	2013-04-05 14:29:21 UTC (rev 55064)
+++ brlcad/trunk/doc/docbook/lessons/en/mged15_creating_a_toy_truck.xml	2013-04-05 14:51:48 UTC (rev 55065)
@@ -146,7 +146,7 @@
 	If you are using <application>BRL-CAD</application> version 6.0 or later, at the Command Window
 	prompt, type:
 
-	<command>inside cab1.s caboff1.s .5 -.1 .7 -.1 -.1 -.1[Enter]</command>
+	<command>inside cab1.s caboff1.s -.1 .7 -.1 -.1 .5 -.1[Enter]</command>
 
 	The inside command tells <application>MGED</application> to:
 	<informaltable>
@@ -156,46 +156,60 @@
 		<entry>inside</entry>
 		<entry>cab1.s</entry>
 		<entry>caboff1.s</entry>
-		<entry>.5</entry>
 		<entry>-.1</entry>
-		<entry>7</entry>
+		<entry>.7</entry>
 		<entry>-.1</entry>
 		<entry>-.1</entry>
+		<entry>.5</entry>
 		<entry>-.1</entry>
 	      </row>
 	      <row>
 		<entry>Inside the shape</entry>
 		<entry>named cab1.s,</entry>
 		<entry>create rpp called caboff1.s</entry>
-		<entry>Make face 1234 (bottom) .5 units thick</entry>
-		<entry>Make face 5678 (top) -.1 units thick</entry>
-		<entry>Make face 1485 (rear) .7 units thick</entry>
-		<entry>Make face 2376 (front) -.1 units thick</entry>
-		<entry>Make face 1265 (right) -.1 units thick</entry>
-		<entry>Make face 3478 (left) -.1 units thick</entry>
+		<entry>Make face 1234 (front) -.1 units thick</entry>
+		<entry>Make face 5678 (rear) .7 units thick</entry>
+		<entry>Make face 1485 (right) -.1 units thick</entry>
+		<entry>Make face 2376 (left) -.1 units thick</entry>
+		<entry>Make face 1265 (bottom) .5 units thick</entry>
+		<entry>Make face 3487 (top) -.1 units thick</entry>
 	      </row>
 	    </tbody>
 	  </tgroup>
 	</informaltable>
 
       </para>
+
       <note>
 	<para>
-	  In this example, each negative thickness number indicates that
-	  caboff1.s will protrude through the corresponding face of cab1.s.
+	  In this example, each negative thickness number indicates
+	  that caboff1.s will protrude through the corresponding face
+	  of cab1.s.  Running the l (list) command, you can see the
+	  vertices for the box it creates:
+
+	  <literallayout class="normal">
+caboff1.s:  ARB8
+	  1 (1.1, -0.1, 0.5)
+	  2 (1.1, 1.1, 0.5)
+	  3 (1.1, 1.1, 1.1)
+	  4 (1.1, -0.1, 1.1)
+	  5 (0.7, -0.1, 0.5)
+	  6 (0.7, 1.1, 0.5)
+	  7 (0.7, 1.1, 1.1)
+	  8 (0.7, -0.1, 1.1)
+	  </literallayout>
 	</para>
       </note>
+
       <note>
 	<para>
-	  Note that in <application>BRL-CAD</application> versions 6.0 and later, the faces of an rpp are
-	  numbered in a different order than in previous versions. So, if you
-	  are using a pre-6.0 release of the package, the following parameter
-	  order must be used for the aforementioned inside command:
+	  Note that in older versions
+	  of <application>BRL-CAD</application>, the faces of an rpp
+	  were numbered in a different order.  So, if your results
+	  don't match, kill caboff1.s and run this command instead:
 
-	  <command>inside cab1.s caboff1.s -.1 .7 -.1 -.1 .5 -.1[Enter]</command>
+	  <command>inside cab1.s caboff1.s .5 -.1 .7 -.1 -.1 -.1[Enter]</command>
 
-	  (The order of faces in this case is front, rear, right, left, bottom,
-	  top instead of that specified for versions 6.0 and later.)
 	</para>
       </note>
 

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.