Merge branch 'master' of https://github.com/euphy/polargraphcontroller

Also mentioned against issue #14.

.gitignore

@@ -5,3 +5,5 @@
 /application.linux32
 /application.linux64
 /application.macosx
+Thumbs.db
+xx_default.properties.txt

ControlFrame.pde

@@ -0,0 +1,52 @@
+// the ControlFrame class extends PApplet, so we 
+// are creating a new processing applet inside a
+// new frame with a controlP5 object loaded
+public class ControlFrame extends PApplet {
+  public int w, h;
+  int abc = 100;
+  public ControlP5 cp5;
+  protected PApplet parent;
+
+  private ControlFrame() {
+  }
+
+  public ControlFrame(PApplet theParent, int theWidth, int theHeight) {
+    this.parent = theParent;
+    this.w = theWidth;
+    this.h = theHeight;
+  }
+
+  public ControlP5 cp5() {
+    if (this.cp5 == null) {
+      this.cp5 = this.setupControlP5();
+    }
+    return this.cp5;
+  }
+
+  public PApplet getParent() {
+    return this.parent;
+  }
+
+  public void setup() {
+    size(w, h);
+    frameRate(5);
+  }
+
+  public ControlP5 setupControlP5() {
+    println("About to create new ControlP5");
+    ControlP5 cp5 = new ControlP5(this);
+    println("Created: " + cp5);
+    while (cp5 == null) {
+      println("Was null: " + cp5);
+    }
+    println("Finally created: " + cp5);
+    return cp5;
+  }
+
+  public void draw() {
+    background(abc);
+  }
+}
+
+
+

ControlFrameSimple.pde

@@ -0,0 +1,40 @@
+
+// the ControlFrame class extends PApplet, so we 
+// are creating a new processing applet inside a
+// new frame with a controlP5 object loaded
+public class ControlFrameSimple extends PApplet {
+
+  int w, h;
+
+  int bg;
+
+  public void setup() {
+    size(w, h);
+    frameRate(5);
+    cp5 = new ControlP5( this );
+  }
+
+  public void draw() {
+    background( bg );
+  }
+
+  private ControlFrameSimple() {
+  }
+
+  public ControlFrameSimple(Object theParent, int theWidth, int theHeight, int theColor) {
+    parent = theParent;
+    w = theWidth;
+    h = theHeight;
+    bg = theColor;
+  }
+
+
+  public ControlP5 cp5() {
+    return this.cp5;
+  }
+
+  ControlP5 cp5;
+
+  Object parent;
+}
+

DisplayMachine.pde

@@ -1,31 +1,31 @@
 /**
- Polargraph controller
- Copyright Sandy Noble 2012.
- 
- This file is part of Polargraph Controller.
- 
- Polargraph Controller is free software: you can redistribute it and/or modify
- it under the terms of the GNU General Public License as published by
- the Free Software Foundation, either version 3 of the License, or
- (at your option) any later version.
- 
- Polargraph Controller is distributed in the hope that it will be useful,
- but WITHOUT ANY WARRANTY; without even the implied warranty of
- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- GNU General Public License for more details.
- 
- You should have received a copy of the GNU General Public License
- along with Polargraph Controller.  If not, see <http://www.gnu.org/licenses/>.
- 
- Requires the excellent ControlP5 GUI library available from http://www.sojamo.de/libraries/controlP5/.
- Requires the excellent Geomerative library available from http://www.ricardmarxer.com/geomerative/.
- 
- This is an application for controlling a polargraph machine, communicating using ASCII command language over a serial link.
- 
- sandy.noble@gmail.com
- http://www.polargraph.co.uk/
- http://code.google.com/p/polargraph/
- */
+  Polargraph controller
+  Copyright Sandy Noble 2015.
+
+  This file is part of Polargraph Controller.
+
+  Polargraph Controller is free software: you can redistribute it and/or modify
+  it under the terms of the GNU General Public License as published by
+  the Free Software Foundation, either version 3 of the License, or
+  (at your option) any later version.
+
+  Polargraph Controller is distributed in the hope that it will be useful,
+  but WITHOUT ANY WARRANTY; without even the implied warranty of
+  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+  GNU General Public License for more details.
+
+  You should have received a copy of the GNU General Public License
+  along with Polargraph Controller.  If not, see <http://www.gnu.org/licenses/>.
+    
+  Requires the excellent ControlP5 GUI library available from http://www.sojamo.de/libraries/controlP5/.
+  Requires the excellent Geomerative library available from http://www.ricardmarxer.com/geomerative/.
+  
+  This is an application for controlling a polargraph machine, communicating using ASCII command language over a serial link.
+
+  sandy.noble@gmail.com
+  http://www.polargraph.co.uk/
+  https://github.com/euphy/polargraphcontroller
+*/
 
 class DisplayMachine extends Machine
 {
@@ -316,6 +316,10 @@ class DisplayMachine extends Machine
     {
       drawExtractedPixelCentres();
     }
+    if (displayingGridSpots)
+    {
+      drawGridIntersections();
+    }
     if (displayingDensityPreview)
     {
       drawExtractedPixelDensities();
@@ -519,21 +523,37 @@ class DisplayMachine extends Machine
       {
         if (pointPaths[i] != null) 
         {
-          beginShape();
+          boolean inShape = false;
           for (int j = 0; j<pointPaths[i].length; j++)
           {
             PVector p = new PVector(pointPaths[i][j].x, pointPaths[i][j].y);
             p = PVector.mult(p, scaling);
             p = PVector.add(p, position);
-            if (getPage().surrounds(inSteps(p)))
+            if (getPictureFrame().surrounds(inSteps(p)))
             {
+              if (!inShape) 
+              {
+                beginShape();
+                inShape = true;
+              }
+//              PVector nativeCoords = asNativeCoords(inSteps(p));
+//              println(j + "!  Adding point " + nativeCoords);
+              
               p = scaleToScreen(p);
               stroke(strokeColour);
               vertex(p.x, p.y);
-              //ellipse(p.x, p.y, 3, 3);
+//              ellipse(p.x, p.y, 2, 2);
+            }
+            else
+            {
+              if (inShape) 
+              {
+                endShape();
+                inShape = false;
+              }
             }
           }
-          endShape();
+          if (inShape) endShape();
         }
       }
       if (drawCentroid)
@@ -647,10 +667,18 @@ class DisplayMachine extends Machine
    */
   public void drawRows()
   {
-    PVector mVect = getMouseVector();
-
+    float rowThickness = inMM(getGridSize()) * getScaling();
+    rowThickness = (rowThickness < 1.0) ? 1.0 : rowThickness;
+    strokeWeight(rowThickness);
+    stroke(150, 200, 255, 50);
+    strokeCap(SQUARE);
+    drawRow(getMouseVector(), true, true);
+    noStroke();
+  }
+  
+  public void drawRow(PVector mouse, boolean left, boolean right) {
     // scale it to  find out the coordinates on the machine that the mouse is pointing at.
-    mVect = scaleToDisplayMachine(mVect);
+    PVector mVect = scaleToDisplayMachine(mouse);
     // convert it to the native coordinates system
     mVect = convertToNative(mVect);
     // snap it to the grid
@@ -661,17 +689,15 @@ class DisplayMachine extends Machine
     // and finally, because scaleToScreen also allows for the machine position (offset), subtract it.
     mVect.sub(getOffset());
 
-    float rowThickness = inMM(getGridSize()) * getScaling();
-    rowThickness = (rowThickness < 1.0) ? 1.0 : rowThickness;
-    strokeWeight(rowThickness);
-    stroke(150, 200, 255, 50);
-    strokeCap(SQUARE);
-
     float dia = mVect.x*2;
-    arc(getOutline().getLeft(), getOutline().getTop(), dia, dia, 0, 1.57079633);
+    if (left) {
+      arc(getOutline().getLeft(), getOutline().getTop(), dia, dia, 0, 1.57079633);
+    }
 
     dia = mVect.y*2;
-    arc(getOutline().getRight(), getOutline().getTop(), dia, dia, 1.57079633, 3.14159266);
+    if (right) {
+      arc(getOutline().getRight(), getOutline().getTop(), dia, dia, 1.57079633, 3.14159266);
+    }
     
   }
 
@@ -688,6 +714,28 @@ class DisplayMachine extends Machine
       line(scaledPos.x-1, scaledPos.y+1, scaledPos.x+1, scaledPos.y-1);
     }
   }
+  
+  void drawGridIntersections()
+  {
+//    println("oh");
+  }
+  
+  int pixel_maxDensity(float penSize, float rowSizeInMM) 
+  {
+    float numberOfSegments = rowSizeInMM / penSize;
+
+    int maxDens = 1;
+    
+    if (numberOfSegments >= 2.0) {
+      maxDens = int(numberOfSegments);
+    }
+  
+    if (maxDens <= 1) {
+      maxDens = 1;
+    }  
+    
+    return maxDens;
+  }
 
   void drawExtractedPixelDensities()
   {
@@ -696,27 +744,62 @@ class DisplayMachine extends Machine
     pixelSize = (pixelSize < 1.0) ? 1.0 : pixelSize;
 
     pixelSize = pixelSize * getPixelScalingOverGridSize();
+    
+    float rowSizeInMM = inMM(getGridSize()) * getPixelScalingOverGridSize();
+    
+    int posterizeLevels = 255;
+    
+    if (previewPixelDensityRange) {
+      posterizeLevels = pixel_maxDensity(currentPenWidth, rowSizeInMM);
+    }
+    else {
+      posterizeLevels = densityPreviewPosterize;
+    }
 
     if (getExtractedPixels() != null)
     {
       for (PVector cartesianPos : getExtractedPixels())
       {
-        if ((cartesianPos.z <= pixelExtractBrightThreshold) && (cartesianPos.z >= pixelExtractDarkThreshold))
+        if ((cartesianPos.z <= pixelExtractBrightThreshold) && 
+            (cartesianPos.z >= pixelExtractDarkThreshold))
         {
           // scale em, danno.
           PVector scaledPos = scaleToScreen(cartesianPos);
           noStroke();
-          fill(cartesianPos.z);
+          if ((scaledPos.x <= 0) || (scaledPos.x > windowWidth) ||
+              (scaledPos.y <= 0) || (scaledPos.y > windowHeight)) {
+              continue;
+            }  
+          
+          // Posterize the density value
+          int reduced = int(map(cartesianPos.z, 1, 255, 1, posterizeLevels)+0.5);
+          int brightness = int(map(reduced, 1, posterizeLevels, 1, 255));
+          
+          fill(brightness);
           switch (getDensityPreviewStyle())
           {
             case DENSITY_PREVIEW_ROUND: 
-              previewRoundPixel(scaledPos, pixelSize, pixelSize);
+              previewRoundPixel(scaledPos, pixelSize);
+              break;
+            case DENSITY_PREVIEW_ROUND_SIZE:
+              fill(0);
+              previewRoundPixel(scaledPos, map(brightness, 1, posterizeLevels, pixelSize, 1));
               break;
             case DENSITY_PREVIEW_DIAMOND:
-              previewDiamondPixel(scaledPos, pixelSize, pixelSize, cartesianPos.z);
+              previewDiamondPixel(scaledPos, pixelSize, pixelSize, brightness);
               break;
+            case DENSITY_PREVIEW_NATIVE:
+              previewNativePixel(scaledPos, pixelSize, brightness);
+              break; 
+            case DENSITY_PREVIEW_NATIVE_SIZE:
+              previewNativePixel(scaledPos, map(brightness, 1, posterizeLevels, pixelSize, 1), 50);
+              break; 
+            case DENSITY_PREVIEW_NATIVE_ARC:
+              previewRoundPixel(scaledPos, pixelSize*0.8);
+              previewNativeArcPixel(scaledPos, pixelSize, brightness);
+              break; 
             default:
-              previewRoundPixel(scaledPos, pixelSize, pixelSize);
+              previewRoundPixel(scaledPos, pixelSize);
               break;
           }
         }
@@ -736,17 +819,139 @@ class DisplayMachine extends Machine
     quad(pos.x, pos.y-halfHeight, pos.x+halfWidth, pos.y, pos.x, pos.y+halfHeight, pos.x-halfWidth, pos.y);
     
   }
-  void previewNativePixel(PVector pos, float wide, float high)
+  
+  void previewNativePixel(PVector pos, float size, float brightness)
   {
-    // shall I try and draw a diamond here instead? OK! I'll do it! Ha!
-    float halfWidth = wide / 2.0;
-    float halfHeight = high / 2.0;
-    quad(pos.x, pos.y-halfHeight, pos.x+halfWidth, pos.y, pos.x, pos.y+halfHeight, pos.x-halfWidth, pos.y);
+    float half = size / 2.0;
     
+    // arcs from the left-hand corner
+    float distFromPointA = getOutline().getTopLeft().dist(pos);
+    float distFromPointB = getOutline().getTopRight().dist(pos);
+
+    List<PVector> int1 = findIntersections(getOutline().getLeft(), distFromPointA-half, getOutline().getRight(), distFromPointB-half, size);
+    List<PVector> int2 = findIntersections(getOutline().getLeft(), distFromPointA+half, getOutline().getRight(), distFromPointB-half, size);
+    
+    if (!int1.isEmpty() && !int2.isEmpty()) {
+      fill(0,0,0, 255-brightness);
+      beginShape();
+  
+      // plot out the vertexes    
+      vertex(int1.get(0).x, int1.get(0).y);
+      vertex(int2.get(0).x, int2.get(0).y);
+      vertex(int2.get(1).x, int2.get(1).y);
+      vertex(int1.get(1).x, int1.get(1).y);
+      vertex(int1.get(0).x, int1.get(0).y);
+      endShape();
+    }
   }
-  void previewRoundPixel(PVector pos, float wide, float high)
+
+  void previewNativeArcPixel(PVector pos, float size, float brightness)
   {
-     ellipse(pos.x, pos.y, wide*1.1, high*1.1);
+    float half = size / 2.0;
+//    fill(0,0,0, 255-brightness);
+    beginShape();
+    
+    // arcs from the left-hand corner
+    float distFromPointA = getOutline().getTopLeft().dist(pos);
+    float distFromPointB = getOutline().getTopRight().dist(pos);
+
+    List<PVector> int1 = findIntersections(getOutline().getLeft(), distFromPointA-half, getOutline().getRight(), distFromPointB-half, size);
+    List<PVector> int2 = findIntersections(getOutline().getLeft(), distFromPointA+half, getOutline().getRight(), distFromPointB-half, size);
+
+    // plot out the vertexes    
+    noFill();
+    stroke(0,0,0, 255-brightness);
+    try {
+    
+      float i1Angle1 = atan2(int1.get(0).y-getOutline().getTop(), int1.get(0).x-getOutline().getLeft());
+      float i1Angle2 = atan2(int1.get(1).y-getOutline().getTop(), int1.get(1).x-getOutline().getLeft());
+      arc(getOutline().getLeft(), getOutline().getTop(), (distFromPointA-half)*2, (distFromPointA-half)*2, i1Angle1, i1Angle2);
+  
+      i1Angle1 = atan2(int2.get(0).y-getOutline().getTop(), int2.get(0).x-getOutline().getLeft());
+      i1Angle2 = atan2(int2.get(1).y-getOutline().getTop(), int2.get(1).x-getOutline().getLeft());
+      arc(getOutline().getLeft(), getOutline().getTop(), (distFromPointA+half)*2, (distFromPointA+half)*2, i1Angle1, i1Angle2);
+  
+      i1Angle1 = atan2( int1.get(0).y-getOutline().getTop(), int1.get(0).x-getOutline().getRight());
+      i1Angle2 = atan2( int2.get(0).y-getOutline().getTop(), int2.get(0).x-getOutline().getRight());
+      arc(getOutline().getRight(), getOutline().getTop(), (distFromPointB-half)*2, (distFromPointB-half)*2, i1Angle2, i1Angle1);
+  
+      i1Angle1 = atan2( int1.get(1).y-getOutline().getTop(), int1.get(1).x-getOutline().getRight());
+      i1Angle2 = atan2( int2.get(1).y-getOutline().getTop(), int2.get(1).x-getOutline().getRight());
+      arc(getOutline().getRight(), getOutline().getTop(), (distFromPointB+half)*2, (distFromPointB+half)*2, i1Angle2, i1Angle1);
+    }
+    catch (IndexOutOfBoundsException ioobe) {
+      println(ioobe);
+    }
+    finally {
+      endShape();
+    }
+  }
+  
+  
+        
+
+  void previewRoundPixel(PVector pos, float dia)
+  {
+     ellipse(pos.x, pos.y, dia*1.1, dia*1.1);
+  }
+  
+  // compute and draw intersections
+  /**
+  circle1 = c1x is the centre, and r1 is the radius of the arc to be drawn.
+  circle2 = c2x, r2 describe the arc that is used to calculate the start and 
+  end point of the drawn arc.
+  
+  circle3 = c2x, r3 is calculated by adding size to r2.
+  
+  The drawn arc should start at the intersection with the circle1, 
+  and end at the intersection with circle3.
+  
+  The clever bits of this are nicked off http://processing.org/discourse/beta/num_1223494826.html
+  */
+  List<PVector> findIntersections(float c1x, float r1, float c2x, float r2, float pixelSize)
+  {
+    float c1y = getOutline().getTop();
+    float c2y = getOutline().getTop();
+    float d=getOutline().getWidth(); // distance between centers
+    float base1, h1, base2, h2; // auxiliary distances
+    //  p, middle point between q1 and q2
+    // q1 dn q2 intersection points
+    float p1x,p1y,p2x,p2y, q1x,q1y,q2x,q2y; 
+  
+    if(d<abs(r1-r2) || d>r1+r2)
+    {
+      println("C1 and C2 do not intersect");
+      return new ArrayList();
+    }
+    else if(d==r1+r2)
+    { // outside each other, intersect in one point
+      return new ArrayList();
+    }
+    else
+    {
+      // intersect in two points
+      base1 = (r1*r1-r2*r2+d*d) / (2*d);
+      h1 = sqrt(r1*r1-base1*base1);
+  
+      p1x = c1x+base1*(c2x-c1x)/d;
+      p1y = c1y+base1*(c2y-c1y)/d;
+      q1x=abs(p1x-h1*(c2y-c1y)/d);
+      q1y=abs(p1y+h1*(c2x-c1x)/d);
+
+      float r3 = r2+pixelSize;
+      base2 = (r1*r1-r3*r3+d*d) / (2*d);
+      h2 = sqrt(r1*r1-base2*base2);
+  
+      p2x = c1x+base2*(c2x-c1x)/d;
+      p2y = c1y+base2*(c2y-c1y)/d;
+      q2x=abs(p2x-h2*(c2y-c1y)/d);
+      q2y=abs(p2y+h2*(c2x-c1x)/d);
+
+      List<PVector> l = new ArrayList<PVector>(2);
+      l.add(new PVector(q1x, q1y));
+      l.add(new PVector(q2x, q2y));
+      return l;
+     }
   }
   
   color getPixelAtScreenCoords(PVector pos)

DrawPixelsWindow.pde

@@ -0,0 +1,138 @@
+///*------------------------------------------------------------------------
+//    Details about the "drawing" subwindow
+//------------------------------------------------------------------------*/
+
+public Integer renderStartDirection = DRAW_DIR_SE; // default start drawing in SE direction (DOWN)
+public Integer renderStartPosition = DRAW_DIR_NE; // default top right hand corner for start
+public Integer renderStyle = PIXEL_STYLE_SQ_FREQ; // default pixel style square wave
+
+ControlFrameSimple addDrawPixelsControlFrame(String theName, int theWidth, int theHeight, int theX, int theY, int theColor ) {
+  final Frame f = new Frame( theName );
+  final ControlFrameSimple p = new ControlFrameSimple( this, theWidth, theHeight, theColor );
+
+  f.add( p );
+  p.init();
+  f.setTitle(theName);
+  f.setSize( p.w, p.h );
+  f.setLocation( theX, theY );
+  f.addWindowListener( new WindowAdapter() {
+    @Override
+      public void windowClosing(WindowEvent we) {
+      p.dispose();
+      f.dispose();
+    }
+  }
+  );
+  f.setResizable( true );
+  f.setVisible( true );
+  // sleep a little bit to allow p to call setup.
+  // otherwise a nullpointerexception might be caused.
+  try {
+    Thread.sleep( 100 );
+  } 
+  catch(Exception e) {
+  }
+  
+  // set up controls
+  RadioButton rPos = p.cp5().addRadioButton("radio_startPosition",10,10)
+    .add("Top-right", DRAW_DIR_NE)
+    .add("Bottom-right", DRAW_DIR_SE)
+    .add("Bottom-left", DRAW_DIR_SW)
+    .add("Top-left", DRAW_DIR_NW)
+    .plugTo(this, "radio_startPosition");
+
+  RadioButton rSkip = p.cp5().addRadioButton("radio_pixelSkipStyle",10,100)
+    .add("Lift pen over masked pixels", 1)
+    .add("Draw masked pixels as blanks", 2)
+    .plugTo(this, "radio_pixelSkipStyle");
+
+  RadioButton rStyle = p.cp5().addRadioButton("radio_pixelStyle",100,10);
+  rStyle.add("Variable frequency square wave", PIXEL_STYLE_SQ_FREQ);
+  rStyle.add("Variable size square wave", PIXEL_STYLE_SQ_SIZE);
+  rStyle.add("Solid square wave", PIXEL_STYLE_SQ_SOLID);
+  rStyle.add("Scribble", PIXEL_STYLE_SCRIBBLE);
+  if (currentHardware >= HARDWARE_VER_MEGA) {
+    rStyle.add("Spiral", PIXEL_STYLE_CIRCLE);
+    rStyle.add("Sawtooth", PIXEL_STYLE_SAW);
+  }
+  rStyle.plugTo(this, "radio_pixelStyle");
+  
+
+  Button submitButton = p.cp5().addButton("submitDrawWindow",0,280,10,120,20)
+    .setLabel("Generate commands")
+    .plugTo(this, "submitDrawWindow");
+      
+  return p;
+}
+
+void radio_startPosition(int pos) {
+  renderStartPosition = pos;
+  radio_rowStartDirection(1);
+}
+
+void radio_rowStartDirection(int dir) {
+  if (renderStartPosition == DRAW_DIR_NE || renderStartPosition == DRAW_DIR_SW)
+    renderStartDirection = (dir == 0) ? DRAW_DIR_NW : DRAW_DIR_SE;
+  else if (renderStartPosition == DRAW_DIR_SE || renderStartPosition == DRAW_DIR_NW)
+    renderStartDirection = (dir == 0) ? DRAW_DIR_NE : DRAW_DIR_SW;
+}
+
+void radio_pixelStyle(int style)  {
+  renderStyle = style;
+}
+
+void radio_pixelSkipStyle(int style) {
+  if (style == 1)
+    liftPenOnMaskedPixels = true;
+  else if (style == 2)
+    liftPenOnMaskedPixels = false;
+}
+
+void submitDrawWindow(int theValue) {
+  println("draw.");
+  println("Style: " + renderStyle);
+  println("Start pos: " + renderStartPosition);
+  println("Start dir: " + renderStartDirection);
+
+  switch (renderStyle) {
+    case PIXEL_STYLE_SQ_FREQ: button_mode_renderSquarePixel(); break;
+    case PIXEL_STYLE_SQ_SIZE: button_mode_renderScaledSquarePixels(); break;
+    case PIXEL_STYLE_SQ_SOLID: button_mode_renderSolidSquarePixels(); break;
+    case PIXEL_STYLE_SCRIBBLE: button_mode_renderScribblePixels(); break;
+    case PIXEL_STYLE_CIRCLE: button_mode_renderCirclePixel(); break;
+    case PIXEL_STYLE_SAW: button_mode_renderSawPixel(); break;
+  }
+}
+
+
+class DrawPixelsWindow extends ControlFrame {
+
+
+	public DrawPixelsWindow () {
+		super(parentPapplet, 450, 150);
+		
+		int xPos = 100;
+		int yPos = 100;
+		String name = DRAW_PIXELS_WINDOW_NAME;
+
+		final Frame f = new Frame(DRAW_PIXELS_WINDOW_NAME);
+		f.add(this);
+		this.init();
+		f.setTitle(CHANGE_SERIAL_PORT_WINDOW_NAME);
+		f.setSize(super.w, super.h);
+		f.setLocation(xPos, yPos);
+		f.setResizable(false);
+		f.setVisible(true);
+
+		f.addWindowListener( new WindowAdapter() {
+			@Override
+			public void windowClosing(WindowEvent we) {
+				f.dispose();
+			}
+		});
+		
+
+	}
+
+
+}

FileLoading.pde

@@ -0,0 +1,439 @@
+/**
+  Polargraph controller
+  Copyright Sandy Noble 2018.
+
+  This file is part of Polargraph Controller.
+
+  Polargraph Controller is free software: you can redistribute it and/or modify
+  it under the terms of the GNU General Public License as published by
+  the Free Software Foundation, either version 3 of the License, or
+  (at your option) any later version.
+
+  Polargraph Controller is distributed in the hope that it will be useful,
+  but WITHOUT ANY WARRANTY; without even the implied warranty of
+  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+  GNU General Public License for more details.
+
+  You should have received a copy of the GNU General Public License
+  along with Polargraph Controller.  If not, see <http://www.gnu.org/licenses/>.
+    
+  Requires the excellent ControlP5 GUI library available from http://www.sojamo.de/libraries/controlP5/.
+  Requires the excellent Geomerative library available from http://www.ricardmarxer.com/geomerative/.
+  
+  This is an application for controlling a polargraph machine, communicating using ASCII command language over a serial link.
+
+  sandy.noble@gmail.com
+  http://www.polargraph.co.uk/
+  https://github.com/euphy/polargraphcontroller
+*/
+
+
+
+void loadImageWithFileChooser()
+{
+  SwingUtilities.invokeLater(new Runnable() 
+  {
+    public void run() {
+      JFileChooser fc = new JFileChooser();
+      if (lastImageDirectory != null) fc.setCurrentDirectory(lastImageDirectory);
+      fc.setFileFilter(new ImageFileFilter());
+      fc.setDialogTitle("Choose an image file...");
+
+      int returned = fc.showOpenDialog(frame);
+      
+      lastImageDirectory = fc.getCurrentDirectory();
+      
+      if (returned == JFileChooser.APPROVE_OPTION) 
+      {
+        File file = fc.getSelectedFile();
+        // see if it's an image
+        PImage img = loadImage(file.getPath());
+        if (img != null) 
+        {
+          img = null;
+          getDisplayMachine().loadNewImageFromFilename(file.getPath());
+          if (getDisplayMachine().pixelsCanBeExtracted() && isBoxSpecified())
+          {
+            getDisplayMachine().extractPixelsFromArea(getBoxVector1(), getBoxVectorSize(), getGridSize(), sampleArea);
+          }
+        }
+      }
+    }
+  });
+}
+
+class ImageFileFilter extends javax.swing.filechooser.FileFilter 
+{
+  public boolean accept(File file) {
+      String filename = file.getName();
+      filename.toLowerCase();
+      if (file.isDirectory() || filename.endsWith(".png") || filename.endsWith(".jpg") || filename.endsWith(".jpeg")) 
+        return true;
+      else
+        return false;
+  }
+  public String getDescription() {
+      return "Image files (PNG or JPG)";
+  }
+}
+
+void loadVectorWithFileChooser()
+{
+  SwingUtilities.invokeLater(new Runnable() 
+  {
+    public void run() {
+      JFileChooser fc = new JFileChooser();
+      if (lastImageDirectory != null) 
+      { 
+        fc.setCurrentDirectory(lastImageDirectory);
+      }
+      
+      fc.setFileFilter(new VectorFileFilter());
+      fc.setDialogTitle("Choose a vector file...");
+      int returned = fc.showOpenDialog(frame);
+      lastImageDirectory = fc.getCurrentDirectory();
+      
+      if (returned == JFileChooser.APPROVE_OPTION) 
+      {
+        File file = fc.getSelectedFile();
+        if (file.exists())
+        {
+          RShape shape = loadShapeFromFile(file.getPath());
+          if (shape != null) 
+          {
+            setVectorFilename(file.getPath());
+            setVectorShape(shape);
+          }
+          else 
+          {
+            println("File not found (" + file.getPath() + ")");
+          }
+        }
+      }
+    }
+  }
+  );
+}
+
+class VectorFileFilter extends javax.swing.filechooser.FileFilter 
+{
+  public boolean accept(File file) {
+    String filename = file.getName();
+    filename.toLowerCase();
+    if (file.isDirectory() || filename.endsWith(".svg") || isGCodeExtension(filename))
+      return true;
+    else
+      return false;
+  }
+  public String getDescription() {
+    return "Vector graphic files (SVG, GCode)";
+  }
+}
+
+void loadNewPropertiesFilenameWithFileChooser()
+{
+  SwingUtilities.invokeLater(new Runnable() 
+  {
+    public void run() 
+    {
+      JFileChooser fc = new JFileChooser();
+      if (lastPropertiesDirectory != null) fc.setCurrentDirectory(lastPropertiesDirectory);
+      fc.setFileFilter(new PropertiesFileFilter());
+      
+      fc.setDialogTitle("Choose a config file...");
+
+      int returned = fc.showOpenDialog(frame);
+      
+      lastPropertiesDirectory = fc.getCurrentDirectory();
+      
+      if (returned == JFileChooser.APPROVE_OPTION) 
+      {
+        File file = fc.getSelectedFile();
+        if (file.exists())
+        {
+          println("New properties file exists.");
+          newPropertiesFilename = file.toString();
+          println("new propertiesFilename: "+  newPropertiesFilename);
+          propertiesFilename = newPropertiesFilename;
+          // clear old properties.
+          props = null;
+          loadFromPropertiesFile();
+          
+          // set values of number spinners etc
+          updateNumberboxValues();
+        }   
+      }
+    }
+  });
+}
+
+class PropertiesFileFilter extends javax.swing.filechooser.FileFilter 
+{
+  public boolean accept(File file) {
+      String filename = file.getName();
+      filename.toLowerCase();
+      if (file.isDirectory() || filename.endsWith(".properties.txt")) 
+        return true;
+      else
+        return false;
+  }
+  public String getDescription() {
+      return "Properties files (*.properties.txt)";
+  }
+}
+
+void saveNewPropertiesFileWithFileChooser()
+{
+  SwingUtilities.invokeLater(new Runnable() 
+  {
+    public void run() 
+    {
+      JFileChooser fc = new JFileChooser();
+      if (lastPropertiesDirectory != null) fc.setCurrentDirectory(lastPropertiesDirectory);
+      fc.setFileFilter(new PropertiesFileFilter());
+      
+      fc.setDialogTitle("Enter a config file name...");
+
+      int returned = fc.showSaveDialog(frame);
+      if (returned == JFileChooser.APPROVE_OPTION) 
+      {
+        File file = fc.getSelectedFile();
+        newPropertiesFilename = file.toString();
+        newPropertiesFilename.toLowerCase();
+        if (!newPropertiesFilename.endsWith(".properties.txt"))
+          newPropertiesFilename+=".properties.txt";
+          
+        println("new propertiesFilename: "+  newPropertiesFilename);
+        propertiesFilename = newPropertiesFilename;
+        savePropertiesFile();
+        // clear old properties.
+        props = null;
+        loadFromPropertiesFile();
+      }
+    }
+  });
+}
+
+
+
+RShape loadShapeFromFile(String filename) {
+  RShape sh = null;
+  if (filename.toLowerCase().endsWith(".svg")) {
+    sh = RG.loadShape(filename);
+  }
+  else if (isGCodeExtension(filename)) {
+    sh = loadShapeFromGCodeFile(filename);
+  }
+  return sh;
+}
+
+
+boolean isGCodeExtension(String filename) {
+  return (filename.toLowerCase().endsWith(".gcode") || filename.toLowerCase().endsWith(".g") || filename.toLowerCase().endsWith(".ngc") || filename.toLowerCase().endsWith(".txt"));
+}
+
+
+int countLines(String filename) throws IOException {
+    InputStream is = new BufferedInputStream(new FileInputStream(filename));
+    try {
+        byte[] c = new byte[1024];
+        int count = 0;
+        int readChars = 0;
+        boolean empty = true;
+        while ((readChars = is.read(c)) != -1) {
+            empty = false;
+            for (int i = 0; i < readChars; ++i) {
+                if (c[i] == '\n') {
+                    ++count;
+                }
+            }
+        }
+        return (count == 0 && !empty) ? 1 : count+1;
+    } finally {
+        is.close();
+    }
+}
+
+RShape loadShapeFromGCodeFile(String filename) {
+  noLoop();
+  RShape parent = null;
+  BufferedReader reader = null;
+  long totalPoints = 0;
+  long time = millis();
+  long countLines = 0;
+
+  try {
+    countLines = countLines(filename);
+    println("" + countLines + " lines found.");
+    if (countLines < 1) {
+      throw new IOException("No lines found in GCode file.");
+    }
+    reader = createReader(filename);
+    parent = new RShape();
+    String line;
+    boolean drawLine = false;
+    int gCodeZAxisChanges = 0;
+    
+    long lineNo = 0;
+    float lastPercent = 0.0f;
+    boolean reportStatus = true;
+    while ((line = reader.readLine ()) != null) {
+      lineNo++;
+//      println("Line: " + line);
+      
+      if (reportStatus) {
+        float percent = ((float)lineNo / (float)countLines) * 100.0;
+        println("----" + percent + "% of the way through.");
+        lastPercent = percent;
+      }
+
+      if (line.toUpperCase().startsWith("G")) {
+        if (reportStatus) {
+          println(new StringBuilder().append(lineNo).append(" of ").append(countLines).append(": ").append(line).append(". Points: ").append(totalPoints).toString());
+          long free = Runtime.getRuntime().freeMemory();
+          long maximum = Runtime.getRuntime().maxMemory();
+          println(new StringBuilder().append("Free: ").append(free).append(", max: ").append(maximum).toString());
+        }
+        
+        Map<String, Float> ins = null;
+        try {
+          ins = unpackGCodeInstruction(line);
+        }
+        catch (Exception e) {
+          println(e.toString());
+          continue;
+        }
+//        println("Ins: " + ins);
+        Integer code = Math.round(ins.get("G"));
+        
+        Float z = ins.get("Z");
+        if (z != null) {
+          gCodeZAxisChanges++;
+          if (gCodeZAxisChanges == 2) {
+            println("Assume second z axis change is to drop the pen to start drawing " + z);
+            gcodeZAxisDrawingHeight = z;
+            drawLine = true;
+          }
+          else if (gCodeZAxisChanges > 2) {
+            drawLine = isGCodeZAxisForDrawing(z);
+          }
+          else {
+            println("Assume first z axis change is to RAISE the pen " + z);
+            drawLine = false;
+          }
+        }
+        else { //  if there is no Z axis, assume it's always on
+//          drawLine = true; // this isn't always safe!
+        }        
+        
+        Float x = ins.get("X");
+        Float y = ins.get("Y");
+        if (x != null && y == null) {
+          // move x axis only, use y of last
+          RPoint[][] points = parent.getPointsInPaths();
+          RPoint rp = points[points.length-1][points[points.length-1].length-1];
+          y = rp.y;
+        }
+        else if (x == null && y != null) {
+          // move y axis only, use x of last
+          RPoint[][] points = parent.getPointsInPaths();
+          RPoint rp = points[points.length-1][points[points.length-1].length-1];
+          x = rp.x;
+        }
+        
+        if (x != null && y != null) {
+          // move both x and y axis
+          if (drawLine) {
+            parent.addLineTo(x, y);
+          }
+          else {
+            parent.addMoveTo(x, y);
+          }
+        }
+      }
+      else {
+        
+      }
+      
+      if ((millis() - time) > 500) {
+        time = millis();
+        reportStatus = true;
+      }
+      else {
+        reportStatus = false;
+      }
+      
+      if (lineNo == (countLines-1)) {
+        reportStatus = true;
+      }
+      
+    }
+  }
+  catch (IOException e) {
+    println("IOExecption reading lines from the gcode file " + filename);
+    e.printStackTrace();
+  }
+  finally {
+    try {
+      reader.close();
+    } 
+    catch (IOException e) {
+      println("IOException closing the gcode file " + filename);
+      e.printStackTrace();
+    }
+  }
+  
+  RPoint[][] points = parent.getPointsInPaths();
+  totalPoints = 0;
+  if (points != null) {
+    for (int i = 0; i<points.length; i++) {
+      if (points[i] != null) {
+        for (int j = 0; j<points[i].length; j++) {
+          totalPoints++;
+        }
+      }
+    }
+  }
+
+  String conclusionMessage = "Imported " + totalPoints + " points from " + countLines + " lines of code in the file.";
+  println(conclusionMessage);
+  javax.swing.JOptionPane.showMessageDialog(null, conclusionMessage);
+
+  loop();
+  return parent;
+}
+
+Boolean isGCodeZAxisForDrawing(float z) {
+  return gcodeZAxisDrawingHeight.compareTo(z) == 0;
+}
+
+Map<String, Float> unpackGCodeInstruction(String line) throws Exception {
+  Map<String, Float> instruction = new HashMap<String, Float>(4);
+  try {
+    String[] splitted = line.trim().split(" ");
+    for (int i = 0; i < splitted.length; i++) {
+      // remove ; character
+      splitted[i] = splitted[i].replace(";", "");
+      String axis = splitted[i].substring(0, 1);
+      String sanitisedValue = splitted[i].substring(1);
+      sanitisedValue = sanitisedValue.replace(",", ".");
+      Float value = Float.parseFloat(sanitisedValue);
+      if ("X".equalsIgnoreCase(axis) || "Y".equalsIgnoreCase(axis) || "Z".equalsIgnoreCase(axis) || "G".equalsIgnoreCase(axis)) {
+        instruction.put(axis.toUpperCase(), value);
+      }
+    }
+//  println("instruction: " + instruction);
+    if (instruction.isEmpty()) {
+      throw new Exception("Empty instruction");
+    }
+  }
+  catch (NumberFormatException nfe) {
+    println("Number format exception: " + nfe.getMessage()); 
+  }
+  catch (Exception e) {
+    println("e: " + e);
+    throw new Exception("Exception while reading the lines from a gcode file: " + line + ", " + e.getMessage());
+  }
+  
+  return instruction;
+}

Machine.pde

@@ -1,6 +1,6 @@
 /**
   Polargraph controller
-  Copyright Sandy Noble 2012.
+  Copyright Sandy Noble 2015.
 
   This file is part of Polargraph Controller.
 
@@ -24,7 +24,7 @@
 
   sandy.noble@gmail.com
   http://www.polargraph.co.uk/
-  http://code.google.com/p/polargraph/
+  https://github.com/euphy/polargraphcontroller
 */
 /**
 *
@@ -39,7 +39,7 @@ class Machine
   protected Rectangle imageFrame = new Rectangle(1500,1500,1000,1000);
   protected Rectangle pictureFrame = new Rectangle(1600,1600,800,800);
 
-  protected Float stepsPerRev = 800.0;
+  protected Float stepsPerRev = 200.0;
   protected Float mmPerRev = 95.0;
   
   protected Float mmPerStep = null;
@@ -186,9 +186,15 @@ class Machine
     return mmInt;
   }
   
+  public float inMMFloat(float steps) 
+  {
+    double mm = steps / getStepsPerMM();
+    return (float) mm;
+  }
+  
   public PVector inMM (PVector steps)
   {
-    PVector mm = new PVector(inMM(steps.x), inMM(steps.y));
+    PVector mm = new PVector(inMMFloat(steps.x), inMMFloat(steps.y));
     return mm;
   }
   
@@ -286,6 +292,16 @@ class Machine
     }
   }
 
+  boolean isMasked(PVector pos, float scalingFactor)
+  {
+    switch (invertMaskMode) {
+      case MASK_IS_UNUSED:  return false;
+      case MASKED_COLOURS_ARE_HIDDEN:  return isChromaKey(pos, scalingFactor);
+      case MASKED_COLOURS_ARE_SHOWN:  return !isChromaKey(pos, scalingFactor);
+      default:  return false;
+    }
+  }
+
   boolean isChromaKey(PVector pos, float scalingFactor)
   {
     if (getImageFrame().surrounds(pos))
@@ -322,8 +338,8 @@ class Machine
   
   public PVector asCartesianCoords(PVector pgCoords)
   {
-    float calcX = int((pow(getWidth(), 2) - pow(pgCoords.y, 2) + pow(pgCoords.x, 2)) / (getWidth()*2));
-    float calcY = int(sqrt(pow(pgCoords.x,2)-pow(calcX,2)));
+    float calcX = (pow(getWidth(), 2.0) - pow(pgCoords.y, 2.0) + pow(pgCoords.x, 2.0)) / (getWidth()*2.0);
+    float calcY = sqrt(pow(pgCoords.x,2.0)-pow(calcX,2.0));
     PVector vect = new PVector(calcX, calcY);
     return vect;
   }
@@ -364,7 +380,7 @@ class Machine
   public void loadDefinitionFromProperties(Properties props)
   {
     // get these first because they are important to convert the rest of them
-    setStepsPerRev(getFloatProperty("machine.motors.stepsPerRev", 800.0));
+    setStepsPerRev(getFloatProperty("machine.motors.stepsPerRev", 200.0));
     setMMPerRev(getFloatProperty("machine.motors.mmPerRev", 95.0));
     
     // now stepsPerMM and mmPerStep should have been calculated. It's safe to get the rest.
@@ -383,12 +399,12 @@ class Machine
     String pos = getStringProperty("controller.page.position.x", "CENTRE");
     float px = 0.0;
     println("machine size: " + getSize().x + ", " + inSteps(pageSize.x));
-    if (pos.equalsIgnoreCase("CENTRE"))
-    {
+    if (pos.equalsIgnoreCase("CENTRE")) {
       px = inMM((getSize().x - pageSize.x) / 2.0);
     }
-    else
+    else {
       px = getFloatProperty("controller.page.position.x", (int) getDisplayMachine().getPageCentrePosition(pageSize.x));
+    }
       
     float py = getFloatProperty("controller.page.position.y", 120);
       
@@ -620,7 +636,7 @@ class Machine
         PVector cartesianCoord = asCartesianCoords(nativeCoord);
         if (selectedArea.surrounds(cartesianCoord))
         {
-          if (isChromaKey(cartesianCoord, scalingFactor))
+          if (isMasked(cartesianCoord, scalingFactor))
           {
             nativeCoord.z = MASKED_PIXEL_BRIGHTNESS; // magic number
             nativeCoords.add(nativeCoord);

MachineExecWindow.pde

@@ -0,0 +1,64 @@
+ControlFrameSimple addMachineExecControlFrame(String theName, int theWidth, int theHeight, int theX, int theY, int theColor ) {
+  final Frame f = new Frame( theName );
+  final ControlFrameSimple p = new ControlFrameSimple( this, theWidth, theHeight, theColor );
+
+  f.add( p );
+  p.init();
+  f.setTitle(theName);
+  f.setSize( p.w, p.h );
+  f.setLocation( theX, theY );
+  f.addWindowListener( new WindowAdapter() {
+    @Override
+      public void windowClosing(WindowEvent we) {
+      p.dispose();
+      f.dispose();
+    }
+  } 
+  );
+  f.setResizable( true );
+  f.setVisible( true );
+  // sleep a little bit to allow p to call setup.
+  // otherwise a nullpointerexception might be caused.
+  try {
+    Thread.sleep( 100 );
+  } 
+  catch(Exception e) {
+  }
+  
+  // set up controls
+  Textfield filenameField = p.cp5().addTextfield("machineExec_execFilename",20,20,150,20)
+    .setText(getStoreFilename())
+    .setLabel("Filename to execute from")
+    .addListener( new ControlListener() {
+      public void controlEvent( ControlEvent ev ) {
+        machineExec_execFilename(ev.getController().getStringValue());
+        Textfield tf = p.cp5().get(Textfield.class, "machineExec_execFilename");
+      }
+    });
+    
+
+  Button submitButton = p.cp5().addButton("machineExec_submitExecFilenameWindow",0,180,20,60,20)
+    .setLabel("Submit")
+    .addListener( new ControlListener() {
+      public void controlEvent( ControlEvent ev ) {
+        p.cp5().get(Textfield.class, "machineExec_execFilename").submit();
+        p.cp5().get(Textfield.class, "machineExec_execFilename").setText(getStoreFilename());
+      }
+    });
+    
+  filenameField.setFocus(true);
+    
+  return p;
+}
+
+void machineExec_execFilename(String filename) {
+  println("Filename event: "+ filename);
+  if (filename != null 
+      && filename.length() <= 12
+      && !"".equals(filename.trim())) {
+    filename = filename.trim();
+    setStoreFilename(filename);
+    sendMachineExecMode();
+  }
+}
+

MachineStoreWindow.pde

@@ -0,0 +1,77 @@
+/*------------------------------------------------------------------------
+    Details about the "machine store" subwindow
+------------------------------------------------------------------------*/
+
+ControlFrameSimple addMachineStoreControlFrame(String theName, int theWidth, int theHeight, int theX, int theY, int theColor ) {
+  final Frame f = new Frame( theName );
+  final ControlFrameSimple p = new ControlFrameSimple( this, theWidth, theHeight, theColor );
+
+  f.add( p );
+  p.init();
+  f.setTitle(theName);
+  f.setSize( p.w, p.h );
+  f.setLocation( theX, theY );
+  f.addWindowListener( new WindowAdapter() {
+    @Override
+      public void windowClosing(WindowEvent we) {
+      p.dispose();
+      f.dispose();
+    }
+  } 
+  );
+  f.setResizable( true );
+  f.setVisible( true );
+  // sleep a little bit to allow p to call setup.
+  // otherwise a nullpointerexception might be caused.
+  try {
+    Thread.sleep( 100 );
+  } 
+  catch(Exception e) {
+  }
+  
+  // set up controls
+
+    Textfield filenameField = p.cp5().addTextfield("machineStore_storeFilename",20,20,150,20)
+      .setText(getStoreFilename())
+      .setLabel("Filename to store to")
+      .addListener( new ControlListener() {
+        public void controlEvent( ControlEvent ev ) {
+          machineStore_storeFilename(ev.getController().getStringValue());
+          Textfield tf = p.cp5().get(Textfield.class, "machineExec_execFilename");
+        }
+      });
+
+    Button submitButton = p.cp5().addButton("machineStore_submitStoreFilenameWindow",0,180,20,60,20)
+      .setLabel("Submit")
+      .addListener( new ControlListener() {
+        public void controlEvent( ControlEvent ev ) {
+          p.cp5().get(Textfield.class, "machineStore_storeFilename").submit();
+          p.cp5().get(Textfield.class, "machineStore_storeFilename").setText(getStoreFilename());
+        }
+      });
+
+    Toggle overwriteToggle = p.cp5().addToggle("machineStore_toggleAppendToFile",true,180,50,20,20)
+      .setCaptionLabel("Overwrite existing file")
+      .plugTo(this, "machineStore_toggleAppendToFile");
+      
+     
+  filenameField.setFocus(true);
+    
+  return p;
+}
+
+void machineStore_toggleAppendToFile(boolean theFlag) {
+  setOverwriteExistingStoreFile(theFlag);
+}
+  
+void machineStore_storeFilename(String filename) {
+  println("Filename event: "+ filename);
+  if (filename != null 
+      && filename.length() <= 12
+      && !"".equals(filename.trim())) {
+    filename = filename.trim();
+    setStoreFilename(filename);
+    sendMachineStoreMode();
+  }
+}
+

Misc.pde

@@ -1,6 +1,6 @@
 /**
   Polargraph controller
-  Copyright Sandy Noble 2012.
+  Copyright Sandy Noble 2015.
 
   This file is part of Polargraph Controller.
 
@@ -24,7 +24,7 @@
 
   sandy.noble@gmail.com
   http://www.polargraph.co.uk/
-  http://code.google.com/p/polargraph/
+  https://github.com/euphy/polargraphcontroller
 */
 
 class Scaler
@@ -52,202 +52,202 @@ class PreviewVector extends PVector
 {
   public String command;
 }
-
-
-import java.awt.Toolkit;
-import java.awt.BorderLayout;
-import java.awt.GraphicsEnvironment;
-
-public class Console extends WindowAdapter implements WindowListener, ActionListener, Runnable
-{
-  private JFrame frame;
-  private JTextArea textArea;
-  private Thread reader;
-  private Thread reader2;
-  private boolean quit;
-
-  private final PipedInputStream pin=new PipedInputStream(); 
-  private final PipedInputStream pin2=new PipedInputStream(); 
-
-  private PrintStream cOut = System.out;
-  private PrintStream cErr = System.err;
-
-  Thread errorThrower; // just for testing (Throws an Exception at this Console
-
-  public Console()
-  {
-    // create all components and add them
-    frame=new JFrame("Java Console");
-    Dimension screenSize=Toolkit.getDefaultToolkit().getScreenSize();
-    Dimension frameSize=new Dimension((int)(screenSize.width/2),(int)(screenSize.height/2));
-    int x=(int)(frameSize.width/2);
-    int y=(int)(frameSize.height/2);
-    frame.setBounds(x,y,frameSize.width,frameSize.height);
-
-    textArea=new JTextArea();
-    textArea.setEditable(false);
-    JButton button=new JButton("clear");
-
-    frame.getContentPane().setLayout(new BorderLayout());
-    frame.getContentPane().add(new JScrollPane(textArea),BorderLayout.CENTER);
-    frame.getContentPane().add(button,BorderLayout.SOUTH);
-    frame.setVisible(true);		
-
-    frame.addWindowListener(this);		
-    button.addActionListener(this);
-
-    try
-    {
-      this.cOut = System.out;
-      PipedOutputStream pout=new PipedOutputStream(this.pin);
-      System.setOut(new PrintStream(pout,true)); 
-    } 
-    catch (java.io.IOException io)
-    {
-      textArea.append("Couldn't redirect STDOUT to this console\n"+io.getMessage());
-    }
-    catch (SecurityException se)
-    {
-      textArea.append("Couldn't redirect STDOUT to this console\n"+se.getMessage());
-    } 
-
-    try 
-    {
-      this.cErr = System.err;
-      PipedOutputStream pout2=new PipedOutputStream(this.pin2);
-      System.setErr(new PrintStream(pout2,true));
-    } 
-    catch (java.io.IOException io)
-    {
-      textArea.append("Couldn't redirect STDERR to this console\n"+io.getMessage());
-    }
-    catch (SecurityException se)
-    {
-      textArea.append("Couldn't redirect STDERR to this console\n"+se.getMessage());
-    } 		
-
-    quit=false; // signals the Threads that they should exit
-
-    // Starting two seperate threads to read from the PipedInputStreams				
-    //
-    reader=new Thread(this);
-    reader.setDaemon(true);	
-    reader.start();	
-    //
-    reader2=new Thread(this);	
-    reader2.setDaemon(true);	
-    reader2.start();
-
-//    // testing part
-//    // you may omit this part for your application
-//    // 
-//    System.out.println("Hello World 2");
-//    System.out.println("All fonts available to Graphic2D:\n");
-//    GraphicsEnvironment ge = GraphicsEnvironment.getLocalGraphicsEnvironment();
-//    String[] fontNames=ge.getAvailableFontFamilyNames();
-//    for(int n=0;n<fontNames.length;n++)  System.out.println(fontNames[n]);		
-//    // Testing part: simple an error thrown anywhere in this JVM will be printed on the Console
-//    // We do it with a seperate Thread becasue we don't wan't to break a Thread used by the Console.
-//    System.out.println("\nLets throw an error on this console");	
-//    errorThrower=new Thread(this);
-//    errorThrower.setDaemon(true);
-//    errorThrower.start();					
-  }
-
-  public synchronized void windowClosed(WindowEvent evt)
-  {
-    quit=true;
-    this.notifyAll(); // stop all threads
-    try { 
-      reader.join(1000);
-      pin.close();
-      System.setOut(this.cOut);
-    } 
-    catch (Exception e){
-    }		
-    try { 
-      reader2.join(1000);
-      pin2.close(); 
-      System.setErr(this.cErr);
-    } 
-    catch (Exception e){
-    }
-//    System.exit(0);
-  }		
-
-  public synchronized void windowClosing(WindowEvent evt)
-  {
-    frame.setVisible(false); // default behaviour of JFrame	
-    frame.dispose();
-  }
-
-  public synchronized void actionPerformed(ActionEvent evt)
-  {
-    textArea.setText("");
-  }
-
-  public synchronized void run()
-  {
-    try
-    {			
-      while (Thread.currentThread()==reader)
-      {
-        try { 
-          this.wait(100);
-        }
-        catch(InterruptedException ie) {
-        }
-        if (pin.available()!=0)
-        {
-          String input=this.readLine(pin);
-          textArea.append(input);
-          textArea.setCaretPosition(textArea.getDocument().getLength());
-
-        }
-        if (quit) return;
-      }
-
-      while (Thread.currentThread()==reader2)
-      {
-        try { 
-          this.wait(100);
-        }
-        catch(InterruptedException ie) {
-        }
-        if (pin2.available()!=0)
-        {
-          String input=this.readLine(pin2);
-          textArea.append(input);
-          textArea.setCaretPosition(textArea.getDocument().getLength());
-
-        }
-        if (quit) return;
-      }			
-    } 
-    catch (Exception e)
-    {
-      textArea.append("\nConsole reports an Internal error.");
-      textArea.append("The error is: "+e);			
-    }
-  }
-  
-  public void close()
-  {
-    this.windowClosing(null);
-  }
-
-  public synchronized String readLine(PipedInputStream in) throws IOException
-  {
-    String input="";
-    do
-    {
-      int available=in.available();
-      if (available==0) break;
-      byte b[]=new byte[available];
-      in.read(b);
-      input=input+new String(b,0,b.length);														
-    }
-    while( !input.endsWith("\n") &&  !input.endsWith("\r\n") && !quit);
-    return input;
-  }				
-}
+//
+//
+//import java.awt.Toolkit;
+//import java.awt.BorderLayout;
+//import java.awt.GraphicsEnvironment;
+//
+//public class Console extends WindowAdapter implements WindowListener, ActionListener, Runnable
+//{
+//  private JFrame frame;
+//  private JTextArea textArea;
+//  private Thread reader;
+//  private Thread reader2;
+//  private boolean quit;
+//
+//  private final PipedInputStream pin=new PipedInputStream(); 
+//  private final PipedInputStream pin2=new PipedInputStream(); 
+//
+//  private PrintStream cOut = System.out;
+//  private PrintStream cErr = System.err;
+//
+//  Thread errorThrower; // just for testing (Throws an Exception at this Console
+//
+//  public Console()
+//  {
+//    // create all components and add them
+//    frame=new JFrame("Java Console");
+//    Dimension screenSize=Toolkit.getDefaultToolkit().getScreenSize();
+//    Dimension frameSize=new Dimension((int)(screenSize.width/2),(int)(screenSize.height/2));
+//    int x=(int)(frameSize.width/2);
+//    int y=(int)(frameSize.height/2);
+//    frame.setBounds(x,y,frameSize.width,frameSize.height);
+//
+//    textArea=new JTextArea();
+//    textArea.setEditable(false);
+//    JButton button=new JButton("clear");
+//
+//    frame.getContentPane().setLayout(new BorderLayout());
+//    frame.getContentPane().add(new JScrollPane(textArea),BorderLayout.CENTER);
+//    frame.getContentPane().add(button,BorderLayout.SOUTH);
+//    frame.setVisible(true);		
+//
+//    frame.addWindowListener(this);		
+//    button.addActionListener(this);
+//
+//    try
+//    {
+//      this.cOut = System.out;
+//      PipedOutputStream pout=new PipedOutputStream(this.pin);
+//      System.setOut(new PrintStream(pout,true)); 
+//    } 
+//    catch (java.io.IOException io)
+//    {
+//      textArea.append("Couldn't redirect STDOUT to this console\n"+io.getMessage());
+//    }
+//    catch (SecurityException se)
+//    {
+//      textArea.append("Couldn't redirect STDOUT to this console\n"+se.getMessage());
+//    } 
+//
+//    try 
+//    {
+//      this.cErr = System.err;
+//      PipedOutputStream pout2=new PipedOutputStream(this.pin2);
+//      System.setErr(new PrintStream(pout2,true));
+//    } 
+//    catch (java.io.IOException io)
+//    {
+//      textArea.append("Couldn't redirect STDERR to this console\n"+io.getMessage());
+//    }
+//    catch (SecurityException se)
+//    {
+//      textArea.append("Couldn't redirect STDERR to this console\n"+se.getMessage());
+//    } 		
+//
+//    quit=false; // signals the Threads that they should exit
+//
+//    // Starting two seperate threads to read from the PipedInputStreams				
+//    //
+//    reader=new Thread(this);
+//    reader.setDaemon(true);	
+//    reader.start();	
+//    //
+//    reader2=new Thread(this);	
+//    reader2.setDaemon(true);	
+//    reader2.start();
+//
+////    // testing part
+////    // you may omit this part for your application
+////    // 
+////    System.out.println("Hello World 2");
+////    System.out.println("All fonts available to Graphic2D:\n");
+////    GraphicsEnvironment ge = GraphicsEnvironment.getLocalGraphicsEnvironment();
+////    String[] fontNames=ge.getAvailableFontFamilyNames();
+////    for(int n=0;n<fontNames.length;n++)  System.out.println(fontNames[n]);		
+////    // Testing part: simple an error thrown anywhere in this JVM will be printed on the Console
+////    // We do it with a seperate Thread becasue we don't wan't to break a Thread used by the Console.
+////    System.out.println("\nLets throw an error on this console");	
+////    errorThrower=new Thread(this);
+////    errorThrower.setDaemon(true);
+////    errorThrower.start();					
+//  }
+//
+//  public synchronized void windowClosed(WindowEvent evt)
+//  {
+//    quit=true;
+//    this.notifyAll(); // stop all threads
+//    try { 
+//      reader.join(1000);
+//      pin.close();
+//      System.setOut(this.cOut);
+//    } 
+//    catch (Exception e){
+//    }		
+//    try { 
+//      reader2.join(1000);
+//      pin2.close(); 
+//      System.setErr(this.cErr);
+//    } 
+//    catch (Exception e){
+//    }
+////    System.exit(0);
+//  }		
+//
+//  public synchronized void windowClosing(WindowEvent evt)
+//  {
+//    frame.setVisible(false); // default behaviour of JFrame	
+//    frame.dispose();
+//  }
+//
+//  public synchronized void actionPerformed(ActionEvent evt)
+//  {
+//    textArea.setText("");
+//  }
+//
+//  public synchronized void run()
+//  {
+//    try
+//    {			
+//      while (Thread.currentThread()==reader)
+//      {
+//        try { 
+//          this.wait(100);
+//        }
+//        catch(InterruptedException ie) {
+//        }
+//        if (pin.available()!=0)
+//        {
+//          String input=this.readLine(pin);
+//          textArea.append(input);
+//          textArea.setCaretPosition(textArea.getDocument().getLength());
+//
+//        }
+//        if (quit) return;
+//      }
+//
+//      while (Thread.currentThread()==reader2)
+//      {
+//        try { 
+//          this.wait(100);
+//        }
+//        catch(InterruptedException ie) {
+//        }
+//        if (pin2.available()!=0)
+//        {
+//          String input=this.readLine(pin2);
+//          textArea.append(input);
+//          textArea.setCaretPosition(textArea.getDocument().getLength());
+//
+//        }
+//        if (quit) return;
+//      }			
+//    } 
+//    catch (Exception e)
+//    {
+//      textArea.append("\nConsole reports an Internal error.");
+//      textArea.append("The error is: "+e);			
+//    }
+//  }
+//  
+//  public void close()
+//  {
+//    this.windowClosing(null);
+//  }
+//
+//  public synchronized String readLine(PipedInputStream in) throws IOException
+//  {
+//    String input="";
+//    do
+//    {
+//      int available=in.available();
+//      if (available==0) break;
+//      byte b[]=new byte[available];
+//      in.read(b);
+//      input=input+new String(b,0,b.length);														
+//    }
+//    while( !input.endsWith("\n") &&  !input.endsWith("\r\n") && !quit);
+//    return input;
+//  }				
+//}

Panel.pde

@@ -24,7 +24,7 @@
 
   sandy.noble@gmail.com
   http://www.polargraph.co.uk/
-  http://code.google.com/p/polargraph/
+  https://github.com/euphy/polargraphcontroller
 */
 class Panel
 {
@@ -120,9 +120,11 @@ class Panel
   
   public void draw()
   {
-//    stroke(outlineColour);
-//    strokeWeight(2);
-//    rect(getOutline().getLeft(), getOutline().getTop(), getOutline().getWidth(), getOutline().getHeight());
+    if (debugPanels) {
+      stroke(outlineColour);
+      strokeWeight(2);
+      rect(getOutline().getLeft(), getOutline().getTop(), getOutline().getWidth(), getOutline().getHeight());
+    }
 
     drawControls();
   }
@@ -131,13 +133,12 @@ class Panel
   {
     for (Controller c : this.getControls())
     {
-      //println("Control: " + c.name());
-      PVector pos = getControlPositions().get(c.name());
+      PVector pos = getControlPositions().get(c.getName());
       float x = pos.x+getOutline().getLeft();
       float y = pos.y+getOutline().getTop();
       c.setPosition(x, y);
       
-      PVector cSize = getControlSizes().get(c.name());
+      PVector cSize = getControlSizes().get(c.getName());
       c.setSize((int)cSize.x, (int)cSize.y);
 
       boolean locked = false;
@@ -146,32 +147,39 @@ class Panel
       
       // any drawing / extracting controls are disabled if there is no selec
       // box specified.
-      if (getControlsToLockIfBoxNotSpecified().contains(c.name()) && !isBoxSpecified())
+      if (getControlsToLockIfBoxNotSpecified().contains(c.getName()) && !isBoxSpecified())
       {
         locked = true;        
       }
       
       // if there is no vector shape loaded then lock the "draw vector"
       // control.
-      if (c.name().equals(MODE_RENDER_VECTORS) && getVectorShape() == null)
+      if (c.getName().equals(MODE_RENDER_VECTORS) && getVectorShape() == null)
       {
         locked = true;
       }
   
       // if there's no image loaded, then hide resizing/moving
-      if (getControlsToLockIfImageNotLoaded().contains(c.name()) && getDisplayMachine().getImage() == null)
+      if (getControlsToLockIfImageNotLoaded().contains(c.getName()) && getDisplayMachine().getImage() == null)
       {
         locked = true;        
       }
+
+      // if there's no vector loaded, then hide vector controls
+      if (getControlsToLockIfVectorNotLoaded().contains(c.getName()) && vectorFilename == null)
+      {
+        locked = true;        
+      }
+
       
-      if (c.name().equals(MODE_LOAD_VECTOR_FILE))
+      if (c.getName().equals(MODE_LOAD_VECTOR_FILE))
       {
         if (getVectorShape() != null)
           c.setLabel("Clear vector");
         else
           c.setLabel("Load vector");
       }
-      else if (c.name().equals(MODE_LOAD_IMAGE))
+      else if (c.getName().equals(MODE_LOAD_IMAGE))
       {
         if (getDisplayMachine().getImage() != null)
           c.setLabel("Clear image");
@@ -200,37 +208,29 @@ class Panel
     }
   }  
   
-  void setHeight(float h)
+  void setSizeByHeight(float h)
   {
+//    println("Setting size for " + this.getName());
     if (this.isResizable())
     {
       if (h <= getMinimumHeight())
         this.getOutline().setHeight(getMinimumHeight());
       else
         this.getOutline().setHeight(h);
+        
       setControlPositions(buildControlPositionsForPanel(this));
       
       float left = 0.0;
-      String controlName = "";
       for (String key : getControlPositions().keySet())
       {
         PVector pos = getControlPositions().get(key);
-        if (pos.x >= left)
+        if (pos.x > left)
         {
           left = pos.x;
-          controlName = key;
         }
       }
-      
-      Map<String, PVector> map = getControlSizes();
-      
-//      PVector size = getControlSizes().get(controlName);
-//      println("size: " + size);
       float right = left + DEFAULT_CONTROL_SIZE.x;
-      
       this.getOutline().setWidth(right);
     }
   }
-  
-  
 }

README.md

@@ -2,19 +2,18 @@ polargraphcontroller
 ====================
 
 Polargraph controller
-Copyright Sandy Noble 2013.
+Copyright Sandy Noble 2018.
 
-- Requires v0.5.4 of the excellent ControlP5 GUI library available from http://www.sojamo.de/libraries/controlP5/ 
-  - (http://code.google.com/p/controlp5/downloads/detail?name=controlP5_0.5.4.zip&can=2&q=).
+- Requires the excellent ControlP5 GUI library available from https://github.com/sojamo/controlp5.
 - Requires the excellent Geomerative library available from http://www.ricardmarxer.com/geomerative/.
-- Currently only runs on Processing v1.5
+- Running on Processing v2.2.1.
 
 This is a desktop application for controlling a polargraph machine, communicating using ASCII command language over a serial link.
 
-Very sorry that this doesn't work with newest versions of the libraries and Processing.
+The [latest releases bundle] (https://github.com/euphy/polargraphcontroller/releases/latest) contains 
+copies of all the libraries that I use, as well as all the source, and compiled versions of the code where sensible.
 
-The zip file at http://code.google.com/p/polargraph/downloads/detail?name=Polargraph 1.7-1.6.zip contains 
-copies of all the libraries.
+How to [run it from source](https://github.com/euphy/polargraph/wiki/Running-the-controller-from-source-code).
 
 sandy.noble@gmail.com
 http://www.polargraph.co.uk/

Rectangle.pde

@@ -1,6 +1,6 @@
 /**
   Polargraph controller
-  Copyright Sandy Noble 2012.
+  Copyright Sandy Noble 2015.
 
   This file is part of Polargraph Controller.
 
@@ -24,7 +24,7 @@
 
   sandy.noble@gmail.com
   http://www.polargraph.co.uk/
-  http://code.google.com/p/polargraph/
+  https://github.com/euphy/polargraphcontroller
 */
 class Rectangle
 {
@@ -75,6 +75,10 @@ class Rectangle
   {
     return getPosition();
   }
+  public PVector getTopRight()
+  {
+    return new PVector(this.size.x+this.position.x, this.position.y);
+  }
   public PVector getBotRight()
   {
     return PVector.add(this.position, this.size);
@@ -118,4 +122,8 @@ class Rectangle
       return false;
   }
   
+  public String toString() {
+	return new StringBuffer().append("Rectangle pos: ").append(this.getPosition()).append(", size: ").append(this.getSize()).append(".").toString();
+  }
+  
 }

SerialPortWindow.pde

@@ -0,0 +1,120 @@
+/*------------------------------------------------------------------------
+ Class and controllers on the "serial port" subwindow
+ ------------------------------------------------------------------------*/
+  
+ControlFrameSimple addSerialPortControlFrame(String theName, int theWidth, int theHeight, int theX, int theY, int theColor ) {
+  final Frame f = new Frame( theName );
+  final ControlFrameSimple p = new ControlFrameSimple( this, theWidth, theHeight, theColor );
+
+  f.add( p );
+  p.init();
+  f.setTitle(theName);
+  f.setSize( p.w, p.h );
+  f.setLocation( theX, theY );
+  f.addWindowListener( new WindowAdapter() {
+    @Override
+      public void windowClosing(WindowEvent we) {
+      p.dispose();
+      f.dispose();
+    }
+  } 
+  );
+  f.setResizable( true );
+  f.setVisible( true );
+  // sleep a little bit to allow p to call setup.
+  // otherwise a nullpointerexception might be caused.
+  try {
+    Thread.sleep( 100 );
+  } 
+  catch(Exception e) {
+  }
+
+  ScrollableList sl = p.cp5().addScrollableList("dropdown_serialPort")
+    .setPosition(10, 10)
+    .setSize(150, 450)
+    .setBarHeight(20)
+    .setItemHeight(16)
+    .plugTo(this, "dropdown_serialPort");  
+
+  sl.addItem("No serial connection", -1);
+
+  String[] ports = Serial.list();
+  
+  for (int i = 0; i < ports.length; i++) {
+    println("Adding " + ports[i]);
+    sl.addItem(ports[i], i);
+  }
+  
+  int portNo = getSerialPortNumber();
+  println("portNo: " + portNo);
+  if (portNo < 0 || portNo >= ports.length)
+    portNo = -1;
+
+  // set the value of the actual control
+  sl.setValue(portNo);
+
+  sl.setOpen(true);
+  return p;
+}
+
+
+void dropdown_serialPort(int newSerialPort) 
+{
+  println("In dropdown_serialPort, newSerialPort: " + newSerialPort);
+
+  // No serial in list is slot 0 in code because of list index
+  // So shift port index by one 
+  newSerialPort -= 1;
+  
+  if (newSerialPort == -2)
+  {
+  } 
+  else if (newSerialPort == -1) {
+    println("Disconnecting serial port.");
+    useSerialPortConnection = false;
+    if (myPort != null)
+    {
+      myPort.stop();
+      myPort = null;
+    }
+    drawbotReady = false;
+    drawbotConnected = false;
+    serialPortNumber = newSerialPort;
+  } 
+  else if (newSerialPort != getSerialPortNumber()) {
+    println("About to connect to serial port in slot " + newSerialPort);
+    // Print a list of the serial ports, for debugging purposes:
+    if (newSerialPort < Serial.list().length) {
+      try {
+        drawbotReady = false;
+        drawbotConnected = false;
+        if (myPort != null) {
+          myPort.stop();
+          myPort = null;
+        }
+        
+        if (getSerialPortNumber() >= 0) 
+          println("closing " + Serial.list()[getSerialPortNumber()]);
+
+        serialPortNumber = newSerialPort;
+        String portName = Serial.list()[serialPortNumber];
+
+        myPort = new Serial(this, portName, getBaudRate());
+        //read bytes into a buffer until you get a linefeed (ASCII 10):
+        myPort.bufferUntil('\n');
+        useSerialPortConnection = true;
+        println("Successfully connected to port " + portName);
+      }
+      catch (Exception e) {
+        println("Attempting to connect to serial port in slot " + getSerialPortNumber() 
+          + " caused an exception: " + e.getMessage());
+      }
+    } else {
+      println("No serial ports found.");
+      useSerialPortConnection = false;
+    }
+  } else {
+    println("no serial port change.");
+  }
+}
+

controlsActions.pde

@@ -1,6 +1,6 @@
 /**
   Polargraph controller
-  Copyright Sandy Noble 2012.
+  Copyright Sandy Noble 2015.
 
   This file is part of Polargraph Controller.
 
@@ -24,9 +24,9 @@
 
   sandy.noble@gmail.com
   http://www.polargraph.co.uk/
-  http://code.google.com/p/polargraph/
+  https://github.com/euphy/polargraphcontroller
 */
-void   button_mode_begin()
+void button_mode_begin()
 {
   button_mode_clearQueue();
 }
@@ -163,12 +163,17 @@ void button_mode_liveConfirmDraw()
     float scaling = getDisplayMachine().inMM(getDisplayMachine().getImageFrame().getWidth()) / captureShape.getWidth();
     PVector position = new PVector(getDisplayMachine().inMM(getDisplayMachine().getImageFrame().getPosition().x), 
     getDisplayMachine().inMM(getDisplayMachine().getImageFrame().getPosition().y));
-  
+
+    int oldPolygonizer = polygonizer;
+    polygonizer = RG.ADAPTATIVE;
+    setupPolygonizer();  
     sendVectorShapes(captureShape, scaling, position, PATH_SORT_CENTRE_FIRST);
     button_mode_penUp();
 
     //  save shape as SVG
     trace_saveShape(captureShape);
+    polygonizer = oldPolygonizer;
+    setupPolygonizer();
   }
 } 
 void toggle_mode_showWebcamRawVideo(boolean flag)
@@ -318,6 +323,7 @@ void button_mode_loadImage()
     getDisplayMachine().setImageFilename(null);
   }
 }
+
 void button_mode_loadVectorFile()
 {
   if (getVectorShape() == null)
@@ -331,6 +337,7 @@ void button_mode_loadVectorFile()
     vectorFilename = null;
   }
 }
+
 void numberbox_mode_pixelBrightThreshold(float value)
 {
   pixelExtractBrightThreshold = (int) value;
@@ -526,12 +533,14 @@ void numberbox_mode_changeMachineWidth(float value)
   clearBoxVectors();
   float steps = getDisplayMachine().inSteps((int) value);
   getDisplayMachine().getSize().x = steps;
+  getDisplayMachine().maxLength = null;
 }
 void numberbox_mode_changeMachineHeight(float value)
 {
   clearBoxVectors();
   float steps = getDisplayMachine().inSteps((int) value);
   getDisplayMachine().getSize().y = steps;
+  getDisplayMachine().maxLength = null;
 }
 void numberbox_mode_changeMMPerRev(float value)
 {
@@ -608,7 +617,7 @@ void button_mode_sendPenWidth()
   NumberFormat nf = NumberFormat.getNumberInstance(Locale.UK);
   DecimalFormat df = (DecimalFormat)nf;  
   df.applyPattern("###.##");
-  addToRealtimeCommandQueue(CMD_CHANGEPENWIDTH+df.format(currentPenWidth)+",END");
+  addToRealtimeCommandQueue(CMD_SETPENWIDTH+df.format(currentPenWidth)+",END");
 }  
 
 void numberbox_mode_changePenTestStartWidth(float value)
@@ -710,4 +719,61 @@ void revertToLastMode()
   currentMode = lastMode;
 }
 
+void button_mode_sendButtonActivate()
+{
+  addToCommandQueue(CMD_ACTIVATE_MACHINE_BUTTON+",END");
+}
+void button_mode_sendButtonDeactivate()
+{
+  addToCommandQueue(CMD_DEACTIVATE_MACHINE_BUTTON+",END");
+}
+
+void numberbox_mode_previewCordOffsetValue(int value)
+{
+  previewCordOffset = value;
+  previewQueue(true);
+}
+
+void dropdown_mode_cycleDensityPreviewStyle(int index) 
+{
+  println("In dropdown_mode_cycleDensityPreviewStyle");
+  densityPreviewStyle = index;
+  println("Style: " + densityPreviewStyle);
+}
+
+void numberbox_mode_changeDensityPreviewPosterize(int value) {
+  if (value < 1) value = 1;
+  else if (value > 255) value = 255;
+  
+  densityPreviewPosterize = value;
+}
+
+void minitoggle_mode_previewPixelDensityRange(boolean flag) {
+  previewPixelDensityRange = flag;
+  println("previewPixelDensityRange: " + previewPixelDensityRange);
+}
+
+void numberbox_mode_changePolygonizerLength(float value) {
+  polygonizerLength = value;
+  setupPolygonizer();
+}
+
+void numberbox_mode_changePolygonizerAdaptativeAngle(float value) {
+  println("numberbox_mode_changePolygonizerAdaptativeAngle");
+  polygonizerAdaptativeAngle = value;
+  setupPolygonizer();
+}
+
+void dropdown_mode_changePolygonizer(int value) 
+{
+  polygonizer = value;
+  setupPolygonizer();
+}
+
+void dropdown_mode_changeMaskInvert(int value)
+{
+  invertMaskMode = value;
+  rebuildPixels();
+}
+
 

controlsActionsWindows.pde

@@ -1,568 +1,412 @@
 /**
-  Polargraph controller
-  Copyright Sandy Noble 2012.
+ Polargraph controller
+ Copyright Sandy Noble 2015.
+ 
+ This file is part of Polargraph Controller.
+ 
+ Polargraph Controller is free software: you can redistribute it and/or modify
+ it under the terms of the GNU General Public License as published by
+ the Free Software Foundation, either version 3 of the License, or
+ (at your option) any later version.
+ 
+ Polargraph Controller is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ GNU General Public License for more details.
+ 
+ You should have received a copy of the GNU General Public License
+ along with Polargraph Controller.  If not, see <http://www.gnu.org/licenses/>.
+ 
+ Requires the excellent ControlP5 GUI library available from http://www.sojamo.de/libraries/controlP5/.
+ Requires the excellent Geomerative library available from http://www.ricardmarxer.com/geomerative/.
+ 
+ This is an application for controlling a polargraph machine, communicating using ASCII command language over a serial link.
+ 
+ sandy.noble@gmail.com
+ http://www.polargraph.co.uk/
+ https://github.com/euphy/polargraphcontroller
+ */
 
-  This file is part of Polargraph Controller.
-
-  Polargraph Controller is free software: you can redistribute it and/or modify
-  it under the terms of the GNU General Public License as published by
-  the Free Software Foundation, either version 3 of the License, or
-  (at your option) any later version.
-
-  Polargraph Controller is distributed in the hope that it will be useful,
-  but WITHOUT ANY WARRANTY; without even the implied warranty of
-  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-  GNU General Public License for more details.
-
-  You should have received a copy of the GNU General Public License
-  along with Polargraph Controller.  If not, see <http://www.gnu.org/licenses/>.
-    
-  Requires the excellent ControlP5 GUI library available from http://www.sojamo.de/libraries/controlP5/.
-  Requires the excellent Geomerative library available from http://www.ricardmarxer.com/geomerative/.
-  
-  This is an application for controlling a polargraph machine, communicating using ASCII command language over a serial link.
-
-  sandy.noble@gmail.com
-  http://www.polargraph.co.uk/
-  http://code.google.com/p/polargraph/
-*/
-
-
-/*------------------------------------------------------------------------
-    Details about the "serial port" subwindow
-------------------------------------------------------------------------*/
-
-void button_mode_serialPortDialog()
-{
-  ControlWindow serialPortWindow = cp5.addControlWindow("changeSerialPortWindow",100,100,150,350);
-  serialPortWindow.hideCoordinates();
-  
-  serialPortWindow.setBackground(getBackgroundColour());
-  Radio r = cp5.addRadio("radio_serialPort",10,10);
-  r.setWindow(serialPortWindow);
-
-  String[] ports = Serial.list();
-  if (getSerialPortNumber() >= 0 && getSerialPortNumber() < ports.length)
-    r.setValue(getSerialPortNumber());
-    
-  r.add("setup", -2);
-  r.add("No serial connection", -1);
-  
-  for (int i = 0; i < ports.length; i++)
-  {
-    r.add(ports[i], i);
-  }
-  
-  int portNo = getSerialPortNumber();
-  if (portNo > -1 && portNo < ports.length)
-    r.activate(ports[portNo]);
-  else
-    r.activate("No serial connection");
-    
-  r.removeItem("setup");
-}
-
-void radio_serialPort(int newSerialPort) 
-{
-  if (newSerialPort == -2)
-  {
-  }
-  else if (newSerialPort == -1)
-  {
-    println("Disconnecting serial port.");
-    useSerialPortConnection = false;
-    if (myPort != null)
-    {
-      myPort.stop();
-      myPort = null;
-    }
-    drawbotReady = false;
-    drawbotConnected = false;
-    serialPortNumber = newSerialPort;
-  }
-  else if (newSerialPort != getSerialPortNumber())
-  {
-    println("About to connect to serial port in slot " + newSerialPort);
-    // Print a list of the serial ports, for debugging purposes:
-    if (newSerialPort < Serial.list().length)
-    {
-      try 
-      {
-        drawbotReady = false;
-        drawbotConnected = false;
-        if (myPort != null)
-        {
-          myPort.stop();
-          myPort = null;
-        }
-        if (getSerialPortNumber() >= 0)
-          println("closing " + Serial.list()[getSerialPortNumber()]);
-        
-        serialPortNumber = newSerialPort;
-        String portName = Serial.list()[serialPortNumber];
-  
-        myPort = new Serial(this, portName, getBaudRate());
-        //read bytes into a buffer until you get a linefeed (ASCII 10):
-        myPort.bufferUntil('\n');
-        useSerialPortConnection = true;
-        println("Successfully connected to port " + portName);
-      }
-      catch (Exception e)
-      {
-        println("Attempting to connect to serial port in slot " + getSerialPortNumber() 
-        + " caused an exception: " + e.getMessage());
-      }
-    }
-    else
-    {
-      println("No serial ports found.");
-      useSerialPortConnection = false;
-    }
-  }
-  else
-  {
-    println("no serial port change.");
-  }
-}
-
-
-/*------------------------------------------------------------------------
-    Details about the "machine store" subwindow
-------------------------------------------------------------------------*/
-
-ControlWindow dialogWindow = null;
-
-void button_mode_machineStoreDialog()
-{
-  this.dialogWindow = cp5.addControlWindow("chooseStoreFilenameWindow",100,100,450,150);
-  dialogWindow.hideCoordinates();
-  
-  dialogWindow.setBackground(getBackgroundColour());
-
-  Textfield filenameField = cp5.addTextfield("storeFilename",20,20,150,20);
-  filenameField.setText(getStoreFilename());
-  filenameField.setLabel("Filename to store to");
-  filenameField.setWindow(dialogWindow);
-
-  Button submitButton = cp5.addButton("submitStoreFilenameWindow",0,180,20,60,20);
-  submitButton.setLabel("Submit");
-  submitButton.setWindow(dialogWindow);
-
-  Toggle overwriteToggle = cp5.addToggle("toggleAppendToFile",true,180,50,20,20);
-  overwriteToggle.setCaptionLabel("Overwrite existing file");
-  overwriteToggle.setWindow(dialogWindow);
-
-  filenameField.setFocus(true);
-
-}
-
-void storeFilename(String filename)
-{
-  println("Filename event: "+ filename);
-  if (filename != null && filename.length() <= 12)
-  {
-    setStoreFilename(filename);
-    sendMachineStoreMode();
-  }
-}
-
-void toggleAppendToFile(boolean theFlag) 
-{
-  setOverwriteExistingStoreFile(theFlag);
-}
-
-void submitStoreFilenameWindow(int theValue) 
-{
-  Textfield tf = (Textfield) cp5.controller("storeFilename");
-  tf.submit();
-}
-
-void button_mode_machineExecDialog()
-{
-  this.dialogWindow = cp5.addControlWindow("chooseExecFilenameWindow",100,100,450,150);
-  dialogWindow.hideCoordinates();
-  
-  dialogWindow.setBackground(getBackgroundColour());
-
-  Textfield filenameField = cp5.addTextfield("execFilename",20,20,150,20);
-  filenameField.setText(getStoreFilename());
-  filenameField.setLabel("Filename to execute from");
-  filenameField.setWindow(dialogWindow);
-
-  Button submitButton = cp5.addButton("submitExecFilenameWindow",0,180,20,60,20);
-  submitButton.setLabel("Submit");
-  submitButton.setWindow(dialogWindow);
-
-  filenameField.setFocus(true);
-
-}
-
-void execFilename(String filename)
-{
-  println("Filename event: "+ filename);
-  if (filename != null && filename.length() <= 12)
-  {
-    setStoreFilename(filename);
-    sendMachineExecMode();
-  }
-}
-void submitExecFilenameWindow(int theValue) 
-{
-  Textfield tf = (Textfield) cp5.controller("execFilename");
-  tf.submit();
-}
-
-void button_mode_sendMachineLiveMode()
-{
+void button_mode_sendMachineLiveMode() {
   sendMachineLiveMode();
 }
 
+String CHANGE_SERIAL_PORT_WINDOW_NAME = "changeSerialPortWindow";
+String MACHINE_STORE_WINDOW_NAME = "chooseStoreFilenameWindow";
+String MACHINE_EXEC_WINDOW_NAME = "chooseExecFilenameWindow";
+String DRAW_PIXELS_WINDOW_NAME = "drawPixelsWindow";
+String DRAW_WRITING_WINDOW_NAME = "drawWritingWindow";
 
+void button_mode_serialPortDialog() {
+  ControlFrameSimple cf = addSerialPortControlFrame("Serial Port", 200, 500, 20, 240, color( 100 ) );
+}
 
+void button_mode_machineStoreDialog() {
+  ControlFrameSimple cf = addMachineStoreControlFrame("Machine Store", 450, 250, 20, 240, color( 100 ) );
+}
 
+void button_mode_machineExecDialog() {
+  ControlFrameSimple cf = addMachineExecControlFrame("Machine Execute", 450, 250, 20, 240, color( 100 ) );
+}
 
-/*------------------------------------------------------------------------
-    Details about the "drawing" subwindow
-------------------------------------------------------------------------*/
-void button_mode_drawPixelsDialog()
-{
-  this.dialogWindow = cp5.addControlWindow("drawPixelsWindow",100,100,450,150);
-  dialogWindow.hideCoordinates();
-  
-  dialogWindow.setBackground(getBackgroundColour());
+void button_mode_drawPixelsDialog() {
+  ControlFrameSimple cf = addDrawPixelsControlFrame("Render pixels", 450, 250, 20, 240, color( 100 ) );
+}
 
-  Radio rPos = cp5.addRadio("radio_startPosition",10,10);
-  rPos.add("Top-right", DRAW_DIR_NE);
-  rPos.add("Bottom-right", DRAW_DIR_SE);
-  rPos.add("Bottom-left", DRAW_DIR_SW);
-  rPos.add("Top-left", DRAW_DIR_NW);
-  rPos.setWindow(dialogWindow);
+void button_mode_drawWritingDialog() {
+  ControlFrameSimple cf = addSpriteWritingControlFrame("Sprite Writing", 450, 250, 20, 240, color( 100 ) );
+}
 
-  Radio rSkip = cp5.addRadio("radio_pixelSkipStyle",10,100);
-  rSkip.add("Lift pen over masked pixels", 1);
-  rSkip.add("Draw masked pixels as blanks", 2);
-  rSkip.setWindow(dialogWindow);
+void button_mode_RandomSpriteDialog() {
+  ControlFrameSimple cf = addRandomSpriteControlFrame("Random Sprite", 450, 250, 20, 240, color( 100 ) );
+}
 
-//  Radio rDir = cp5.addRadio("radio_rowStartDirection",100,10);
-//  rDir.add("Upwards", 0);
-//  rDir.add("Downwards", 1);
-//  rDir.setWindow(dialogWindow);
+void button_mode_drawNorwegianDialog() {
+  ControlFrameSimple cf = addNorwegianPixelControlFrame("Norwegian Pixel", 450, 250, 20, 240, color( 100 ) );
+}
 
-  Radio rStyle = cp5.addRadio("radio_pixelStyle",100,10);
-  rStyle.add("Variable frequency square wave", PIXEL_STYLE_SQ_FREQ);
-  rStyle.add("Variable size square wave", PIXEL_STYLE_SQ_SIZE);
-  rStyle.add("Solid square wave", PIXEL_STYLE_SQ_SOLID);
-  rStyle.add("Scribble", PIXEL_STYLE_SCRIBBLE);
-  if (currentHardware >= HARDWARE_VER_MEGA)
-  {
-    rStyle.add("Spiral", PIXEL_STYLE_CIRCLE);
-    rStyle.add("Sawtooth", PIXEL_STYLE_SAW);
+///*------------------------------------------------------------------------
+//    Details about the "writing" subwindow
+//------------------------------------------------------------------------*/
+
+String spriteWriting_textToWrite = "";
+String spriteWriting_spriteFilePrefix = "sprite/let";
+String spriteWriting_spriteFileSuffix = ".txt";
+
+ControlFrameSimple addSpriteWritingControlFrame(String theName, int theWidth, int theHeight, int theX, int theY, int theColor ) {
+  final Frame f = new Frame( theName );
+  final ControlFrameSimple p = new ControlFrameSimple( this, theWidth, theHeight, theColor );
+
+  f.add( p );
+  p.init();
+  f.setTitle(theName);
+  f.setSize( p.w, p.h );
+  f.setLocation( theX, theY );
+  f.addWindowListener( new WindowAdapter() {
+    @Override
+      public void windowClosing(WindowEvent we) {
+      p.dispose();
+      f.dispose();
+      cp5s.remove(DRAW_WRITING_WINDOW_NAME);
+    }
   }
-  rStyle.setWindow(dialogWindow);
-
-  Button submitButton = cp5.addButton("submitDrawWindow",0,280,10,120,20);
-  submitButton.setLabel("Generate commands");
-  submitButton.setWindow(dialogWindow);
-  
-
-}
-
-public Integer renderStartPosition = DRAW_DIR_NE; // default top right hand corner for start
-public Integer renderStartDirection = DRAW_DIR_SE; // default start drawing in SE direction (DOWN)
-public Integer renderStyle = PIXEL_STYLE_SQ_FREQ; // default pixel style square wave
-void radio_startPosition(int pos)
-{
-  this.renderStartPosition = pos;
-  radio_rowStartDirection(1);
-}
-void radio_rowStartDirection(int dir)
-{
-  if (renderStartPosition == DRAW_DIR_NE || renderStartPosition == DRAW_DIR_SW)
-    renderStartDirection = (dir == 0) ? DRAW_DIR_NW : DRAW_DIR_SE;
-  else if (renderStartPosition == DRAW_DIR_SE || renderStartPosition == DRAW_DIR_NW)
-    renderStartDirection = (dir == 0) ? DRAW_DIR_NE : DRAW_DIR_SW;
-}
-void radio_pixelStyle(int style)
-{
-  renderStyle = style;
-}
-void radio_pixelSkipStyle(int style)
-{
-  if (style == 1)
-    liftPenOnMaskedPixels = true;
-  else if (style == 2)
-    liftPenOnMaskedPixels = false;
-}
-void submitDrawWindow(int theValue) 
-{
-  println("draw.");
-  println("Style: " + renderStyle);
-  println("Start pos: " + renderStartPosition);
-  println("Start dir: " + renderStartDirection);
- 
-  switch (renderStyle)
-  {
-    case PIXEL_STYLE_SQ_FREQ: button_mode_renderSquarePixel(); break;
-    case PIXEL_STYLE_SQ_SIZE: button_mode_renderScaledSquarePixels(); break;
-    case PIXEL_STYLE_SQ_SOLID: button_mode_renderSolidSquarePixels(); break;
-    case PIXEL_STYLE_SCRIBBLE: button_mode_renderScribblePixels(); break;
-    case PIXEL_STYLE_CIRCLE: button_mode_renderCirclePixel(); break;
-    case PIXEL_STYLE_SAW: button_mode_renderSawPixel(); break;
+  );
+  f.setResizable( true );
+  f.setVisible( true );
+  // sleep a little bit to allow p to call setup.
+  // otherwise a nullpointerexception might be caused.
+  try {
+    Thread.sleep( 100 );
+  } 
+  catch(Exception e) {
   }
   
-   
-}
-
-/*------------------------------------------------------------------------
-    Details about the "writing" subwindow
-------------------------------------------------------------------------*/
-String textToWrite = "";
-String spriteFilePrefix = "sprite/let";
-String spriteFileSuffix = ".txt";
-
-void button_mode_drawWritingDialog()
-{
-  this.dialogWindow = cp5.addControlWindow("drawWritingWindow",100,100,450,200);
-  dialogWindow.hideCoordinates();
+  cp5s.put(DRAW_WRITING_WINDOW_NAME, p.cp5());
+  println(cp5s);
   
-  dialogWindow.setBackground(getBackgroundColour());
+  // set up controls
+    Textfield spriteFileField = p.cp5().addTextfield("spriteWriting_spriteFilePrefixField", 20, 20, 150, 20)
+      .setText(spriteWriting_getSpriteFilePrefix())
+      .setLabel("File prefix")
+      .plugTo(this, "spriteWriting_spriteFilePrefixField");
 
-  Textfield spriteFileField = cp5.addTextfield("spriteFilePrefixField",20,20,150,20);
-  spriteFileField.setText(getSpriteFilePrefix());
-  spriteFileField.setLabel("File prefix");
-  spriteFileField.setWindow(dialogWindow);
+    Textfield writingField = p.cp5().addTextfield("spriteWriting_textToWriteField", 20, 60, 400, 20)
+      .setText(spriteWriting_getTextToWrite())
+      .setLabel("Text to write")
+      .plugTo(this, "spriteWriting_textToWriteField");
 
-  Textfield writingField = cp5.addTextfield("textToWriteField",20,60,400,20);
-  writingField.setText(getTextToWrite());
-  writingField.setLabel("Text to write");
-  writingField.setWindow(dialogWindow);
+    Button importTextButton = p.cp5().addButton("spriteWriting_importTextButton", 0, 20, 100, 120, 20)
+      .setLabel("Load text from file")
+      .addListener( new ControlListener() {
+        public void controlEvent( ControlEvent ev ) {
+          spriteWriting_importTextButton();
+        }
+      });
+      
+    RadioButton rPos = p.cp5().addRadioButton("spriteWriting_radio_drawWritingDirection", 20, 140);
+    rPos.add("South-east", DRAW_DIR_SE);
+    rPos.activate("South-east");
+    rPos.plugTo(this, "spriteWriting_radio_drawWritingDirection");
 
-  Button importTextButton = cp5.addButton("importTextButton",0,20,100,120,20);
-  importTextButton.setLabel("Load text from file");
-  importTextButton.setWindow(dialogWindow);
-
-  Radio rPos = cp5.addRadio("radio_drawWritingDirection",20,140);
-//  rPos.add("North-east", DRAW_DIR_NE);
-  rPos.add("South-east", DRAW_DIR_SE);
-//  rPos.add("South-west", DRAW_DIR_SW);
-//  rPos.add("North-west", DRAW_DIR_NW);
-  rPos.setWindow(dialogWindow);
-  
-
-
-  Button submitButton = cp5.addButton("submitWritingWindow",0,300,100,120,20);
-  submitButton.setLabel("Generate commands");
-  submitButton.setWindow(dialogWindow);
-}
-
-void spriteFilePrefixField(String value)
-{
-  spriteFilePrefix = value;
-}
-void textToWriteField(String value)
-{
-  textToWrite = value;
-}
-
-String getTextToWrite()
-{
-  return textToWrite;
-}
-String getSpriteFilePrefix()
-{
-  return spriteFilePrefix;
-}
-String getSpriteFileSuffix()
-{
-  return spriteFileSuffix;
-}
-
-void importTextButton()
-{
-  textToWrite = importTextToWriteFromFile();
-  Textfield tf = (Textfield) cp5.controller("textToWriteField");
-  tf.setText(getTextToWrite());
-  tf.submit();
+    Button submitButton = p.cp5.addButton("spriteWriting_submitWritingWindow", 0, 300, 100, 120, 20)
+      .setLabel("Generate commands")
+      .addListener( new ControlListener() {
+        public void controlEvent( ControlEvent ev ) {
+          spriteWriting_submitWritingWindow(p.cp5());
+        }
+      });
+      
+      
+    return p;
 }
 
 
-void submitWritingWindow(int theValue) 
-{
-  println("Write.");
-  
-  Textfield tf = (Textfield) cp5.controller("spriteFilePrefixField");
-  tf.submit();
-  tf.setText(getSpriteFilePrefix());
-  tf = (Textfield) cp5.controller("textToWriteField");
-  tf.submit();
-  tf.setText(getTextToWrite());
-  
-  println("Start dir: " + renderStartDirection);
-  println("Sprite file prefix: " + spriteFilePrefix);
-  println("Text: " + textToWrite);
 
-  for (int i=0; i<getTextToWrite().length(); i++)
+  void spriteWriting_spriteFilePrefixField(String value) {
+    spriteWriting_spriteFilePrefix = value;
+  }
+  void spriteWriting_textToWriteField(String value) {
+    spriteWriting_textToWrite = value;
+  }
+  String spriteWriting_getTextToWrite() {
+    return spriteWriting_textToWrite;
+  }
+  String spriteWriting_getSpriteFilePrefix() {
+    return spriteWriting_spriteFilePrefix;
+  }
+  String spriteWriting_getSpriteFileSuffix() {
+    return spriteWriting_spriteFileSuffix;
+  }
+
+  void spriteWriting_importTextButton() {
+    println("Text being imported!");
+    selectInput("Select the text file to load the text from:", 
+                "spriteWriting_importTextToWriteFromFile");
+  }
+
+  public void spriteWriting_importTextToWriteFromFile(File selection) {
+    if (selection != null) {
+      String fp = selection.getAbsolutePath();
+      println("Input file: " + fp);
+      List<String> rows = java.util.Arrays.asList(loadStrings(fp));
+      StringBuilder sb = new StringBuilder(200);
+      for (String row : rows) {
+        sb.append(row);
+      }
+      spriteWriting_textToWriteField(sb.toString());
+      println("Completed text import, " + spriteWriting_getTextToWrite().length() + " characters found.");
+
+      println("Text: " + spriteWriting_getTextToWrite());
+      
+      println(cp5s);
+
+      Textfield tf = cp5s.get(DRAW_WRITING_WINDOW_NAME).get(Textfield.class, "spriteWriting_textToWriteField");      
+      if (spriteWriting_getTextToWrite() != null 
+          && !"".equals(spriteWriting_getTextToWrite().trim())) {
+        tf.setText(spriteWriting_getTextToWrite());
+        tf.submit();
+        tf.setText(spriteWriting_getTextToWrite());
+      }
+    }
+  }
+
+  void spriteWriting_submitWritingWindow(ControlP5 parent) 
   {
-    String filename = getSpriteFilePrefix() + (int) getTextToWrite().charAt(i) + getSpriteFileSuffix();
-    addToCommandQueue(CMD_DRAW_SPRITE + int(gridSize * pixelScalingOverGridSize) + "," + filename+",END");
-    println(filename);
+    println("Write.");
+
+    Textfield tf = parent.get(Textfield.class, "spriteWriting_spriteFilePrefixField");
+    tf.submit();
+    tf.setText(spriteWriting_getSpriteFilePrefix());
+
+    Textfield wf = parent.get(Textfield.class, "spriteWriting_textToWriteField");
+    wf.submit();
+    wf.setText(spriteWriting_getTextToWrite());
+
+    println("Start dir: " + renderStartDirection);
+    println("Sprite file prefix: " + spriteWriting_spriteFilePrefix);
+    println("Text: " + spriteWriting_textToWrite);
+
+    for (int i=0; i<spriteWriting_getTextToWrite ().length(); i++) {
+      String filename = spriteWriting_getSpriteFilePrefix() + (int) spriteWriting_getTextToWrite().charAt(i) + spriteWriting_getSpriteFileSuffix();
+      addToCommandQueue(CMD_DRAW_SPRITE + int(gridSize * pixelScalingOverGridSize) + "," + filename+",END");
+      println(filename);
+    }
+  }
+
+
+///*------------------------------------------------------------------------
+//    Details about the "sprite" subwindow
+//------------------------------------------------------------------------*/
+String sprite_spriteFilename;
+int sprite_minSpriteSize = 100;
+int sprite_maxSpriteSize = 500;
+
+ControlFrameSimple addRandomSpriteControlFrame(String theName, int theWidth, int theHeight, int theX, int theY, int theColor ) {
+  final Frame f = new Frame( theName );
+  final ControlFrameSimple p = new ControlFrameSimple( this, theWidth, theHeight, theColor );
+
+  f.add( p );
+  p.init();
+  f.setTitle(theName);
+  f.setSize( p.w, p.h );
+  f.setLocation( theX, theY );
+  f.addWindowListener( new WindowAdapter() {
+    @Override
+      public void windowClosing(WindowEvent we) {
+      p.dispose();
+      f.dispose();
+    }
+  }
+  );
+  f.setResizable( true );
+  f.setVisible( true );
+  // sleep a little bit to allow p to call setup.
+  // otherwise a nullpointerexception might be caused.
+  try {
+    Thread.sleep( 100 );
+  } 
+  catch(Exception e) {
   }
   
+  // set up controls
+
+  Textfield spriteFilenameField = p.cp5().addTextfield("sprite_spriteFilenameField",20,20,400,20)
+    .setText("filename.txt")
+    .setLabel("Sprite filename")
+    .plugTo(this, "sprite_spriteFilenameField");
+
+  Numberbox minSizeField = p.cp5().addNumberbox("sprite_minimumSpriteSize",20,60,100,20)
+    .setValue(sprite_getMinimumSpriteSize())
+    .setMin(10)
+    .setMax(sprite_getMaximumSpriteSize())
+    .setMultiplier(0.5)
+    .setLabel("Minimum size")
+    .plugTo(this, "sprite_minimumSpriteSize");
+
+  Numberbox maxSizeField = p.cp5().addNumberbox("sprite_maximumSpriteSize",20,100,100,20)
+    .setValue(sprite_getMaximumSpriteSize())
+    .setMin(sprite_getMinimumSpriteSize())
+    .setMultiplier(0.5)
+    .setLabel("Maximum size")
+    .plugTo(this, "sprite_maximumSpriteSize");
+
+  Button submitButton = p.cp5().addButton("sprite_submitSpriteWindow",0,300,100,120,20)
+    .setLabel("Draw sprite")
+    .addListener( new ControlListener() {
+      public void controlEvent( ControlEvent ev ) {
+        spriteWriting_submitWritingWindow(p.cp5());
+      }
+    });
+
+  spriteFilenameField.setFocus(true);
+           
+  return p;
 }
 
-
-/*------------------------------------------------------------------------
-    Details about the "sprite" subwindow
-------------------------------------------------------------------------*/
-String spriteFilename;
-int minSpriteSize = 100;
-int maxSpriteSize = 500;
-
-void button_mode_drawSpriteDialog()
+String sprite_getSpriteFilename()
 {
-  this.dialogWindow = cp5.addControlWindow("drawSpriteWindow",100,100,450,200);
-  dialogWindow.hideCoordinates();
-  
-  dialogWindow.setBackground(getBackgroundColour());
-
-  delay(200);
-  Textfield spriteFilenameField = cp5.addTextfield("spriteFilenameField",20,20,400,20);
-  spriteFilenameField.setText("filename.txt");
-  spriteFilenameField.setLabel("Sprite filename");
-  spriteFilenameField.setWindow(dialogWindow);
-
-  Numberbox minSizeField = cp5.addNumberbox("minimumSpriteSize",20,60,100,20);
-  minSizeField.setValue(getMinimumSpriteSize());
-  minSizeField.setMin(10);
-  minSizeField.setMax(getMaximumSpriteSize());
-  minSizeField.setMultiplier(0.5);  
-  minSizeField.setLabel("Minimum size");
-  minSizeField.setWindow(dialogWindow);
-
-  Numberbox maxSizeField = cp5.addNumberbox("maximumSpriteSize",20,100,100,20);
-  maxSizeField.setValue(getMaximumSpriteSize());
-  maxSizeField.setMin(getMinimumSpriteSize());
-  maxSizeField.setMultiplier(0.5);  
-  maxSizeField.setLabel("Maximum size");
-  maxSizeField.setWindow(dialogWindow);
-
-  Radio rPos = cp5.addRadio("radio_drawWritingDirection",20,140);
-  rPos.add("North-east", DRAW_DIR_NE);
-  rPos.add("South-east", DRAW_DIR_SE);
-  rPos.add("South-west", DRAW_DIR_SW);
-  rPos.add("North-west", DRAW_DIR_NW);
-  rPos.setWindow(dialogWindow);
-  
-
-
-  Button submitButton = cp5.addButton("submitSpriteWindow",0,300,100,120,20);
-  submitButton.setLabel("Draw sprite");
-  submitButton.setWindow(dialogWindow);
+  return sprite_spriteFilename;
 }
-
-void radio_drawWritingDirection(int dir)
+int sprite_getMinimumSpriteSize()
 {
-  renderStartDirection = dir;
+  return sprite_minSpriteSize;
+}
+int sprite_getMaximumSpriteSize()
+{
+  return sprite_maxSpriteSize;
 }
 
-String getSpriteFilename()
-{
-  return spriteFilename;
-}
-int getMinimumSpriteSize()
-{
-  return minSpriteSize;
-}
-int getMaximumSpriteSize()
-{
-  return maxSpriteSize;
-}
-
-void submitSpriteWindow(int theValue) 
+void sprite_submitSpriteWindow(Textfield tf) 
 {
   println("Sprite.");
   
-  Textfield tf = (Textfield) cp5.controller("spriteFilenameField");
   tf.submit();
-  tf.setText(getSpriteFilename());
+  tf.setText(sprite_getSpriteFilename());
   
-  println("Start dir: " + renderStartDirection);
-  println("Filename: " + spriteFilename);
+  println("Filename: " + sprite_getSpriteFilename());
 
-  addToCommandQueue(CMD_DRAW_SPRITE + "," + spriteFilename + "," 
-  + getMinimumSpriteSize() + "," + getMaximumSpriteSize() + "," + renderStartDirection + ",END");
+  addToCommandQueue(CMD_DRAW_SPRITE + "," + sprite_getSpriteFilename() + "," 
+  + sprite_getMinimumSpriteSize() + "," + sprite_getMaximumSpriteSize() + "," + DRAW_DIR_NE + ",END");
   
 }
 
 
-/*------------------------------------------------------------------------
-    Details about the "norwegian draw" subwindow
-------------------------------------------------------------------------*/
-String norwegianExecFilename = "filename.pbm";
-int norwegianAmplitude = 20;
-int norwegianWavelength = 2;
+///*------------------------------------------------------------------------
+//    Details about the "norwegian draw" subwindow
+//------------------------------------------------------------------------*/
+String norwegian_execFilename = "filename.pbm";
+int norwegian_amplitude = 20;
+int norwegian_wavelength = 2;
 
-void button_mode_drawNorwegianDialog()
-{
-  this.dialogWindow = cp5.addControlWindow("chooseNorwegianFilenameWindow",100,100,450,150);
-  dialogWindow.hideCoordinates();
+ControlFrameSimple addNorwegianPixelControlFrame(String theName, int theWidth, int theHeight, int theX, int theY, int theColor ) {
+  final Frame f = new Frame( theName );
+  final ControlFrameSimple p = new ControlFrameSimple( this, theWidth, theHeight, theColor );
+
+  f.add( p );
+  p.init();
+  f.setTitle(theName);
+  f.setSize( p.w, p.h );
+  f.setLocation( theX, theY );
+  f.addWindowListener( new WindowAdapter() {
+    @Override
+      public void windowClosing(WindowEvent we) {
+      p.dispose();
+      f.dispose();
+    }
+  }
+  );
+  f.setResizable( true );
+  f.setVisible( true );
+  // sleep a little bit to allow p to call setup.
+  // otherwise a nullpointerexception might be caused.
+  try {
+    Thread.sleep( 100 );
+  } 
+  catch(Exception e) {
+  }
   
-  dialogWindow.setBackground(getBackgroundColour());
+  // set up controls
+  Textfield filenameField = p.cp5().addTextfield("norwegian_execFilename",20,20,150,20)
+    .setText(norwegian_execFilename)
+    .setLabel("Filename to execute from")
+    .plugTo(this, "norwegian_execFilename");
 
-  Textfield filenameField = cp5.addTextfield("norwegianExecFilename",20,20,150,20);
-  filenameField.setText(norwegianExecFilename);
-  filenameField.setLabel("Filename to execute from");
-  filenameField.setWindow(dialogWindow);
+  Numberbox minSizeField = p.cp5().addNumberbox("norwegian_amplitude",20,60,100,20)
+    .setValue(norwegian_amplitude)
+    .setMin(10)
+    .setMultiplier(0.5)  
+    .setLabel("Amplitude")
+    .plugTo(this, "norwegian_amplitude");
 
-  Numberbox minSizeField = cp5.addNumberbox("norwegianAmplitude",20,60,100,20);
-  minSizeField.setValue(norwegianAmplitude);
-  minSizeField.setMin(10);
-  minSizeField.setMultiplier(0.5);  
-  minSizeField.setLabel("Amplitude");
-  minSizeField.setWindow(dialogWindow);
+  Numberbox maxSizeField = p.cp5().addNumberbox("norwegian_wavelength",20,100,100,20)
+  .setValue(norwegian_wavelength)
+  .setMin(1)
+  .setMultiplier(0.5)  
+  .setLabel("Wavelength")
+  .plugTo(this, "norwegian_wavelength");
 
-  Numberbox maxSizeField = cp5.addNumberbox("norwegianWavelength",20,100,100,20);
-  maxSizeField.setValue(norwegianWavelength);
-  maxSizeField.setMin(1);
-  maxSizeField.setMultiplier(0.5);  
-  maxSizeField.setLabel("Wavelength");
-  maxSizeField.setWindow(dialogWindow);
+  Button outlineButton = p.cp5().addButton("norwegian_submitNorwegianExecTraceOutline",0,180,20,80,20)
+  .setLabel("Trace outline")
+  .addListener( new ControlListener() {
+    public void controlEvent( ControlEvent ev ) {
+      norwegian_submitNorwegianExec(p.cp5().get(Textfield.class, "norwegian_execFilename"), true);
+    }
+  });
 
-  Button outlineButton = cp5.addButton("submitNorwegianExecTraceOutline",0,180,20,80,20);
-  outlineButton.setLabel("Trace outline");
-  outlineButton.setWindow(dialogWindow);
-
-  Button submitButton = cp5.addButton("submitNorwegianExecFilenameWindow",0,180,100,80,20);
-  submitButton.setLabel("Submit");
-  submitButton.setWindow(dialogWindow);
+  Button submitButton = p.cp5().addButton("norwegian_submitNorwegianExecFilenameWindow",0,180,100,80,20)
+  .setLabel("Submit")
+  .addListener( new ControlListener() {
+    public void controlEvent( ControlEvent ev ) {
+      norwegian_submitNorwegianExec(p.cp5().get(Textfield.class, "norwegian_execFilename"), false);
+    }
+  });
 
   filenameField.setFocus(true);
-
+      
+      
+  return p;
 }
 
-void submitNorwegianExecTraceOutline(int theValue) 
+
+void norwegian_submitNorwegianExec(Textfield tf, boolean outline)
 {
-  Textfield tf = (Textfield) cp5.controller("norwegianExecFilename");
   tf.submit();
-  tf.setText(norwegianExecFilename);
+  tf.setText(norwegian_execFilename);
   
-  println("Filename:" + norwegianExecFilename);
+  println("Filename:" + norwegian_execFilename);
   
-  addToCommandQueue(CMD_DRAW_NORWEGIAN_OUTLINE + norwegianExecFilename + ",END");
-}
-
-void submitNorwegianExecFilenameWindow(int theValue) 
-{
-  Textfield tf = (Textfield) cp5.controller("norwegianExecFilename");
-  tf.submit();
-  tf.setText(norwegianExecFilename);
-  
-  println("Filename:" + norwegianExecFilename);
-  println("Amplitude:" + norwegianAmplitude);
-  println("Wavelength:" + norwegianWavelength);
-  
-  addToCommandQueue(CMD_DRAW_NORWEGIAN + norwegianExecFilename + ","+norwegianAmplitude+","+norwegianWavelength+",END");
+  if (outline) {
+    addToCommandQueue(CMD_DRAW_NORWEGIAN_OUTLINE + norwegian_execFilename + ",END");
+  }
+  else {
+    println("Filename:" + norwegian_execFilename);
+    println("Amplitude:" + norwegian_amplitude);
+    println("Wavelength:" + norwegian_wavelength);
+    
+    addToCommandQueue(CMD_DRAW_NORWEGIAN + norwegian_execFilename + ","+norwegian_amplitude+","+norwegian_wavelength+",END");
+  }
 }
 
 
 
+

controlsSetup.pde

@@ -1,82 +1,73 @@
 /**
- Polargraph controller
- Copyright Sandy Noble 2012.
- 
- This file is part of Polargraph Controller.
- 
- Polargraph Controller is free software: you can redistribute it and/or modify
- it under the terms of the GNU General Public License as published by
- the Free Software Foundation, either version 3 of the License, or
- (at your option) any later version.
- 
- Polargraph Controller is distributed in the hope that it will be useful,
- but WITHOUT ANY WARRANTY; without even the implied warranty of
- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- GNU General Public License for more details.
- 
- You should have received a copy of the GNU General Public License
- along with Polargraph Controller.  If not, see <http://www.gnu.org/licenses/>.
- 
- Requires the excellent ControlP5 GUI library available from http://www.sojamo.de/libraries/controlP5/.
- Requires the excellent Geomerative library available from http://www.ricardmarxer.com/geomerative/.
- 
- This is an application for controlling a polargraph machine, communicating using ASCII command language over a serial link.
- 
- sandy.noble@gmail.com
- http://www.polargraph.co.uk/
- http://code.google.com/p/polargraph/
- */
-Set<String> getPanelNames()
-{
+  Polargraph controller
+  Copyright Sandy Noble 2015.
+
+  This file is part of Polargraph Controller.
+
+  Polargraph Controller is free software: you can redistribute it and/or modify
+  it under the terms of the GNU General Public License as published by
+  the Free Software Foundation, either version 3 of the License, or
+  (at your option) any later version.
+
+  Polargraph Controller is distributed in the hope that it will be useful,
+  but WITHOUT ANY WARRANTY; without even the implied warranty of
+  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+  GNU General Public License for more details.
+
+  You should have received a copy of the GNU General Public License
+  along with Polargraph Controller.  If not, see <http://www.gnu.org/licenses/>.
+    
+  Requires the excellent ControlP5 GUI library available from http://www.sojamo.de/libraries/controlP5/.
+  Requires the excellent Geomerative library available from http://www.ricardmarxer.com/geomerative/.
+  
+  This is an application for controlling a polargraph machine, communicating using ASCII command language over a serial link.
+
+  sandy.noble@gmail.com
+  http://www.polargraph.co.uk/
+  https://github.com/euphy/polargraphcontroller
+*/
+Set<String> getPanelNames() {
   if (this.panelNames == null)
     this.panelNames = buildPanelNames();
   return this.panelNames;
 }
-List<String> getTabNames()
-{
+List<String> getTabNames() {
   if (this.tabNames == null)
     this.tabNames = buildTabNames();
   return this.tabNames;
 }
-Set<String> getControlNames()
-{
+Set<String> getControlNames() {
   if (this.controlNames == null)
     this.controlNames = buildControlNames();
   return this.controlNames;
 }
-Map<String, List<Controller>> getControlsForPanels()
-{
+Map<String, List<Controller>> getControlsForPanels() {
   if (this.controlsForPanels == null)
     this.controlsForPanels = buildControlsForPanels();
   return this.controlsForPanels;
 }
-Map<String, Controller> getAllControls()
-{
+Map<String, Controller> getAllControls() {
   if (this.allControls == null)
     this.allControls = buildAllControls();
   return this.allControls;
 }
-Map<String, String> getControlLabels()
-{
+Map<String, String> getControlLabels() {
   if (this.controlLabels == null)
     this.controlLabels = buildControlLabels();
   return this.controlLabels;
 }
-Map<String, Set<Panel>> getPanelsForTabs()
-{
+Map<String, Set<Panel>> getPanelsForTabs() {
   if (this.panelsForTabs == null)
     this.panelsForTabs = buildPanelsForTabs();
   return this.panelsForTabs;
 }
-Map<String, Panel> getPanels()
-{
+Map<String, Panel> getPanels() {
   if (this.panels == null)
     this.panels = buildPanels();
   return this.panels;
 }
 
-Set<String> getControlsToLockIfBoxNotSpecified()
-{
+Set<String> getControlsToLockIfBoxNotSpecified() {
   if (this.controlsToLockIfBoxNotSpecified == null)
   {
     this.controlsToLockIfBoxNotSpecified = buildControlsToLockIfBoxNotSpecified();
@@ -84,8 +75,7 @@ Set<String> getControlsToLockIfBoxNotSpecified()
   return this.controlsToLockIfBoxNotSpecified;
 }
 
-Set<String> getControlsToLockIfImageNotLoaded()
-{
+Set<String> getControlsToLockIfImageNotLoaded() {
   if (this.controlsToLockIfImageNotLoaded == null)
   {
     this.controlsToLockIfImageNotLoaded = buildControlsToLockIfImageNotLoaded();
@@ -93,9 +83,15 @@ Set<String> getControlsToLockIfImageNotLoaded()
   return this.controlsToLockIfImageNotLoaded;
 }
 
+Set<String> getControlsToLockIfVectorNotLoaded() {
+  if (this.controlsToLockIfVectorNotLoaded == null)
+  {
+    this.controlsToLockIfVectorNotLoaded = buildControlsToLockIfVectorNotLoaded();
+  }
+  return this.controlsToLockIfVectorNotLoaded;
+}
 
-void hideAllControls()
-{
+void hideAllControls() {
   for (String key : allControls.keySet())
   {
     Controller c = allControls.get(key);
@@ -103,14 +99,13 @@ void hideAllControls()
   }
 }
 
-Map<String, Panel> buildPanels()
-{
+Map<String, Panel> buildPanels() {
   Map<String, Panel> panels = new HashMap<String, Panel>();
 
   float panelHeight = frame.getHeight() - getMainPanelPosition().y - (DEFAULT_CONTROL_SIZE.y*3);
-  Rectangle panelOutline = new Rectangle(getMainPanelPosition(), 
+  Rectangle panelOutlineInput = new Rectangle(getMainPanelPosition(), 
   new PVector((DEFAULT_CONTROL_SIZE.x+CONTROL_SPACING.x)*2, panelHeight));
-  Panel inputPanel = new Panel(PANEL_NAME_INPUT, panelOutline);
+  Panel inputPanel = new Panel(PANEL_NAME_INPUT, panelOutlineInput);
   inputPanel.setResizable(true);
   inputPanel.setOutlineColour(color(200, 200, 200));
   // get controls
@@ -120,8 +115,10 @@ Map<String, Panel> buildPanels()
   inputPanel.setControlSizes(buildControlSizesForPanel(inputPanel));
   panels.put(PANEL_NAME_INPUT, inputPanel);
 
-  Panel rovingPanel = new Panel(PANEL_NAME_ROVING, panelOutline);
-  rovingPanel.setOutlineColour(color(200,200,200));
+  Rectangle panelOutlineRoving = new Rectangle(getMainPanelPosition(), 
+  new PVector((DEFAULT_CONTROL_SIZE.x+CONTROL_SPACING.x)*2, panelHeight));
+  Panel rovingPanel = new Panel(PANEL_NAME_ROVING, panelOutlineRoving);
+  rovingPanel.setOutlineColour(color(100,200,200));
   // get controls
   rovingPanel.setResizable(true);
   rovingPanel.setControls(getControlsForPanels().get(PANEL_NAME_ROVING));
@@ -130,8 +127,10 @@ Map<String, Panel> buildPanels()
   rovingPanel.setControlSizes(buildControlSizesForPanel(rovingPanel));
   panels.put(PANEL_NAME_ROVING, rovingPanel);
 
-  Panel tracePanel = new Panel(PANEL_NAME_TRACE, panelOutline);
-  tracePanel.setOutlineColour(color(200,200,200));
+  Rectangle panelOutlineTrace = new Rectangle(getMainPanelPosition(), 
+  new PVector((DEFAULT_CONTROL_SIZE.x+CONTROL_SPACING.x)*2, panelHeight));
+  Panel tracePanel = new Panel(PANEL_NAME_TRACE, panelOutlineTrace);
+  tracePanel.setOutlineColour(color(200,255,200));
   // get controls
   tracePanel.setResizable(true);
   tracePanel.setControls(getControlsForPanels().get(PANEL_NAME_TRACE));
@@ -140,8 +139,10 @@ Map<String, Panel> buildPanels()
   tracePanel.setControlSizes(buildControlSizesForPanel(tracePanel));
   panels.put(PANEL_NAME_TRACE, tracePanel);
 
-  Panel detailsPanel = new Panel(PANEL_NAME_DETAILS, panelOutline);
-  detailsPanel.setOutlineColour(color(200, 200, 200));
+  Rectangle panelOutlineDetails = new Rectangle(getMainPanelPosition(), 
+  new PVector((DEFAULT_CONTROL_SIZE.x+CONTROL_SPACING.x)*2, panelHeight));
+  Panel detailsPanel = new Panel(PANEL_NAME_DETAILS, panelOutlineDetails);
+  detailsPanel.setOutlineColour(color(200, 200, 255));
   // get controls
   detailsPanel.setResizable(true);
   detailsPanel.setControls(getControlsForPanels().get(PANEL_NAME_DETAILS));
@@ -150,8 +151,10 @@ Map<String, Panel> buildPanels()
   detailsPanel.setControlSizes(buildControlSizesForPanel(detailsPanel));
   panels.put(PANEL_NAME_DETAILS, detailsPanel);
 
-  Panel queuePanel = new Panel(PANEL_NAME_QUEUE, panelOutline);
-  queuePanel.setOutlineColour(color(200, 200, 200));
+  Rectangle panelOutlineQueue = new Rectangle(getMainPanelPosition(), 
+  new PVector((DEFAULT_CONTROL_SIZE.x+CONTROL_SPACING.x)*2, panelHeight));
+  Panel queuePanel = new Panel(PANEL_NAME_QUEUE, panelOutlineQueue);
+  queuePanel.setOutlineColour(color(200, 200, 50));
   // get controls
   queuePanel.setResizable(true);
   queuePanel.setControls(getControlsForPanels().get(PANEL_NAME_QUEUE));
@@ -160,18 +163,20 @@ Map<String, Panel> buildPanels()
   queuePanel.setControlSizes(buildControlSizesForPanel(queuePanel));
   panels.put(PANEL_NAME_QUEUE, queuePanel);
 
-  panelOutline = new Rectangle(
+  Rectangle panelOutlineGeneral = new Rectangle(
     new PVector(getMainPanelPosition().x, getMainPanelPosition().y-((DEFAULT_CONTROL_SIZE.y+CONTROL_SPACING.y)*2)), 
     new PVector((DEFAULT_CONTROL_SIZE.x+CONTROL_SPACING.x)*2, (DEFAULT_CONTROL_SIZE.y+CONTROL_SPACING.y)*2));
-  Panel generalPanel = new Panel(PANEL_NAME_GENERAL, panelOutline);
+  Panel generalPanel = new Panel(PANEL_NAME_GENERAL, panelOutlineGeneral);
   generalPanel.setResizable(false);
-  generalPanel.setOutlineColour(color(200, 200, 200));
+  generalPanel.setOutlineColour(color(200, 50, 200));
   // get controls
   generalPanel.setControls(getControlsForPanels().get(PANEL_NAME_GENERAL));
   // get control positions
   generalPanel.setControlPositions(buildControlPositionsForPanel(generalPanel));
   generalPanel.setControlSizes(buildControlSizesForPanel(generalPanel));
   panels.put(PANEL_NAME_GENERAL, generalPanel);
+  
+  
 
   return panels;
 }
@@ -186,19 +191,31 @@ void updateNumberboxValues()
   initialiseNumberboxValues(getAllControls());
 }
 
+void initConsoleWindow() {
+  consoleArea = cp5.addTextarea("txt")
+                   .setPosition(300, 100)
+                   .setSize(400, 600)
+                   .setFont(createFont("", 12))
+                   .setLineHeight(14)
+                   .setColor(color(255))
+                   .setColorBackground(color(0, 200))
+                   .setColorForeground(color(255, 100))
+                   .setVisible(false);
+}
+
 Set<String> buildControlsToLockIfBoxNotSpecified()
 {
   Set<String> result = new HashSet<String>();
   result.add(MODE_DRAW_OUTLINE_BOX);
-  result.add(MODE_DRAW_OUTLINE_BOX_ROWS);
-  result.add(MODE_DRAW_SHADE_BOX_ROWS_PIXELS);
+//  result.add(MODE_DRAW_OUTLINE_BOX_ROWS);
+//  result.add(MODE_DRAW_SHADE_BOX_ROWS_PIXELS);
   result.add(MODE_RENDER_SQUARE_PIXELS);
   result.add(MODE_RENDER_SCALED_SQUARE_PIXELS);
   result.add(MODE_RENDER_SAW_PIXELS);
   result.add(MODE_RENDER_CIRCLE_PIXELS);
   result.add(MODE_RENDER_PIXEL_DIALOG);
-  result.add(MODE_DRAW_GRID);
-  result.add(MODE_DRAW_TESTPATTERN);
+//  result.add(MODE_DRAW_GRID);
+//  result.add(MODE_DRAW_TESTPATTERN);
   result.add(MODE_RENDER_SOLID_SQUARE_PIXELS);
   result.add(MODE_RENDER_SCRIBBLE_PIXELS);
   result.add(MODE_CONVERT_BOX_TO_PICTUREFRAME);
@@ -218,11 +235,41 @@ Set<String> buildControlsToLockIfImageNotLoaded()
   result.add(MODE_CHANGE_SAMPLE_AREA);
   result.add(MODE_SELECT_PICTUREFRAME);
 
+  result.add(MODE_CHANGE_PIXEL_SCALING);
+  result.add(MODE_CHOOSE_CHROMA_KEY_COLOUR);
+
   return result;
 }
 
+Set<String> buildControlsToLockIfVectorNotLoaded()
+{
+  Set<String> result = new HashSet<String>();
+  result.add(MODE_CHANGE_MIN_VECTOR_LINE_LENGTH);
+  result.add(MODE_RESIZE_VECTOR);
+  result.add(MODE_MOVE_VECTOR);
+  result.add(MODE_CHANGE_POLYGONIZER_LENGTH);
+  result.add(MODE_CHANGE_POLYGONIZER);
+
+  return result;
+}
+
+CallbackListener toFront = new CallbackListener() {
+  public void controlEvent(CallbackEvent theEvent) {
+      theEvent.getController().bringToFront();
+      ((ScrollableList)theEvent.getController()).open();
+  }
+};
+
+CallbackListener close = new CallbackListener() {
+  public void controlEvent(CallbackEvent theEvent) {
+      ((ScrollableList)theEvent.getController()).close();
+  }
+};
+
 Map<String, Controller> buildAllControls()
 {
+  
+  initConsoleWindow();
 
   Map<String, Controller> map = new HashMap<String, Controller>();
 
@@ -233,6 +280,8 @@ Map<String, Controller> buildAllControls()
       Button b = cp5.addButton(controlName, 0, 100, 100, 100, 100);
       b.setLabel(getControlLabels().get(controlName));
       b.hide();
+      controlP5.Label l = b.getCaptionLabel();
+      l.align(ControlP5.LEFT, CENTER);
       map.put(controlName, b);
       //      println("Added button " + controlName);
     }
@@ -241,9 +290,9 @@ Map<String, Controller> buildAllControls()
       Toggle t = cp5.addToggle(controlName, false, 100, 100, 100, 100);
       t.setLabel(getControlLabels().get(controlName));
       t.hide();
-      controlP5.Label l = t.captionLabel();
-      l.style().marginTop = -17; //move upwards (relative to button size)
-      l.style().marginLeft = 4; //move to the right
+      controlP5.Label l = t.getCaptionLabel();
+      l.align(ControlP5.LEFT, CENTER);
+      l.getStyle().setPaddingLeft(4);
       map.put(controlName, t);
       //      println("Added toggle " + controlName);
     }
@@ -252,9 +301,9 @@ Map<String, Controller> buildAllControls()
       Toggle t = cp5.addToggle(controlName, false, 100, 100, 100, 100);
       t.setLabel(getControlLabels().get(controlName));
       t.hide();
-      controlP5.Label l = t.captionLabel();
-      l.style().marginTop = -17; //move upwards (relative to button size)
-      l.style().marginLeft = 4; //move to the right
+      controlP5.Label l = t.getCaptionLabel();
+      l.align(ControlP5.LEFT, CENTER);
+      l.getStyle().setPaddingLeft(4);
       map.put(controlName, t);
       //      println("Added minitoggle " + controlName);
     }
@@ -264,21 +313,57 @@ Map<String, Controller> buildAllControls()
       n.setLabel(getControlLabels().get(controlName));
       n.hide();
       n.setDecimalPrecision(0);
-      controlP5.Label l = n.captionLabel();
-      l.style().marginTop = -17; //move upwards (relative to button size)
-      l.style().marginLeft = 40; //move to the right
+      controlP5.Label l = n.getCaptionLabel();
+      l.align(ControlP5.LEFT, CENTER);
+      l.getStyle().setPaddingLeft(35);
       // change the control direction to left/right
       n.setDirection(Controller.VERTICAL);
       map.put(controlName, n);
       //      println("Added numberbox " + controlName);
     }
+    else if (controlName.startsWith("dropdown_"))
+    {
+      ScrollableList sl = cp5.addScrollableList(controlName, 100, 100, 100, 100);
+      sl.setBarHeight(20);
+      sl.setItemHeight(20);
+      sl.setLabel(getControlLabels().get(controlName));
+      sl.setType(ScrollableList.DROPDOWN);
+      sl.onEnter(toFront);
+      sl.onLeave(close);
+      sl.setHeight(100);
+      sl.hide();
+      
+      controlP5.Label l = sl.getCaptionLabel();
+      map.put(controlName, sl);
+      println("Added dropdown " + controlName);
+    }
   }
 
+  initialiseButtonValues(map);
   initialiseToggleValues(map);
   initialiseNumberboxValues(map);
+  initialiseDropdownContents(map);
+  initialiseDropdownValues(map);
   return map;
 }
 
+Map<String, Controller> initialiseButtonValues(Map<String, Controller> map)
+{
+  for (String key : map.keySet())
+  {
+    if (key.startsWith("button_"))
+    {
+      Button n = (Button) map.get(key);
+      if (MODE_CHANGE_POLYGONIZER.equals(key)) {
+        n.setValue(polygonizer);
+        n.setLabel(this.controlLabels.get(MODE_CHANGE_POLYGONIZER) + ": " + polygonizer);              
+      }        
+    }
+  }
+  
+  return map;
+}
+        
 Map<String, Controller> initialiseNumberboxValues(Map<String, Controller> map)
 {
   for (String key : map.keySet())
@@ -443,7 +528,7 @@ Map<String, Controller> initialiseNumberboxValues(Map<String, Controller> map)
         n.setDecimalPrecision(2);
         n.setValue(pixelScalingOverGridSize);
         n.setMin(0.1);
-        n.setMax(4.0);
+//        n.setMax(4.0);
         n.setMultiplier(0.01);
       }
       else if (MODE_CHANGE_MIN_VECTOR_LINE_LENGTH.equals(key))
@@ -500,6 +585,33 @@ Map<String, Controller> initialiseNumberboxValues(Map<String, Controller> map)
         n.setMax(PATH_LENGTH_HIGHPASS_CUTOFF_MAX);
         n.setMultiplier(0.5);
       }
+      else if (MODE_ADJUST_PREVIEW_CORD_OFFSET.equals(key))
+      {
+        n.setDecimalPrecision(2);
+        n.setValue(0.0);
+        n.setMultiplier(0.5);
+      }
+      else if (MODE_CHANGE_DENSITY_PREVIEW_POSTERIZE.equals(key))
+      {
+        n.setValue(densityPreviewPosterize);
+        n.setMin(1);
+        n.setMax(255);
+        n.setDecimalPrecision(1);
+        n.setMultiplier(0.1);
+      }
+      else if (MODE_CHANGE_POLYGONIZER_LENGTH.equals(key)) {
+        n.setValue(polygonizerLength);
+        n.setMin(1.0);
+        n.setDecimalPrecision(1);
+        n.setMultiplier(0.1);
+      }
+      else if (MODE_CHANGE_POLYGONIZER_ADAPTATIVE_ANGLE.equals(key)) {
+        n.setValue(polygonizerAdaptativeAngle);
+        n.setMin(0.0);
+        n.setMax(1.57079632679);
+        n.setDecimalPrecision(2);
+        n.setMultiplier(0.01);
+      }
     }
   }
   return map;
@@ -550,12 +662,71 @@ Map<String, Controller> initialiseToggleValues(Map<String, Controller> map)
       Toggle t = (Toggle) map.get(key);
       t.setValue((rotateWebcamImage) ? 1 : 0);
     }
-    
+    else if (MODE_PREVIEW_PIXEL_DENSITY_RANGE.equals(key))
+    {
+      Toggle t = (Toggle) map.get(key);
+      t.setValue((previewPixelDensityRange) ? 1 : 0);
+    }
   }
   return map;
 }
 
+Map<String, Controller> initialiseDropdownContents(Map<String, Controller> map)
+{
+  println("Init dropdown contents");
+  for (String key : map.keySet())
+  {
+    if (MODE_CYCLE_DENSITY_PREVIEW_STYLE.equals(key))
+    {
+      println("Adding " + key);
+      ScrollableList sl = (ScrollableList) map.get(key);
+      sl.setItems(densityPreviewStyles);
+    }
+    if (MODE_CHANGE_POLYGONIZER.equals(key))
+    {
+      println("Adding " + key);
+      ScrollableList sl = (ScrollableList) map.get(key);
+      sl.setItems(polygonizerStyles);
+    }
+    if (MODE_CHANGE_INVERT_MASK.equals(key))
+    {
+      println("Adding " + key);
+      ScrollableList sl = (ScrollableList) map.get(key);
+      sl.setItems(invertMaskModes);
+    }
+  }
+  return map;
+}
 
+Map<String, Controller> initialiseDropdownValues(Map<String, Controller> map)
+{
+  println("Init dropdown values");
+  for (String key : map.keySet())
+  {
+    if (MODE_CYCLE_DENSITY_PREVIEW_STYLE.equals(key))
+    {
+      println("Adding " + key);
+      ScrollableList sl = (ScrollableList) map.get(key);
+      sl.setValue(densityPreviewStyle);
+      sl.close();
+    }
+    else if (MODE_CHANGE_POLYGONIZER.equals(key))
+    {
+      println("Adding " + key);
+      ScrollableList sl = (ScrollableList) map.get(key);
+      sl.setValue(polygonizer);
+      sl.close();
+    }
+    else if (MODE_CHANGE_INVERT_MASK.equals(key))
+    {
+      println("Adding " + key);
+      ScrollableList sl = (ScrollableList) map.get(key);
+      sl.setValue(polygonizer);
+      sl.close();
+    }
+  }
+  return map;
+}
 
 
 String getControlLabel(String butName)
@@ -574,10 +745,23 @@ Map<String, PVector> buildControlPositionsForPanel(Panel panel)
   int row = 0;
   for (Controller controller : panel.getControls())
   {
-    if (controller.name().startsWith("minitoggle_"))
+    
+    if (controller.getName().startsWith("minitoggle_"))
     {
       PVector p = new PVector(col*(DEFAULT_CONTROL_SIZE.x+CONTROL_SPACING.x), row*(DEFAULT_CONTROL_SIZE.y+CONTROL_SPACING.y));
-      map.put(controller.name(), p);
+      map.put(controller.getName(), p);
+      row++;
+      if (p.y + (DEFAULT_CONTROL_SIZE.y*2) >= panel.getOutline().getHeight())
+      {
+        row = 0;
+        col++;
+      }
+    }
+    else if (controller.getName().startsWith("dropdown_"))
+    {
+      PVector p = new PVector(col*(DEFAULT_CONTROL_SIZE.x+CONTROL_SPACING.x), row*(DEFAULT_CONTROL_SIZE.y+CONTROL_SPACING.y));
+      println(controller);
+      map.put(controller.getName(), p);
       row++;
       if (p.y + (DEFAULT_CONTROL_SIZE.y*2) >= panel.getOutline().getHeight())
       {
@@ -588,7 +772,7 @@ Map<String, PVector> buildControlPositionsForPanel(Panel panel)
     else
     {
       PVector p = new PVector(col*(DEFAULT_CONTROL_SIZE.x+CONTROL_SPACING.x), row*(DEFAULT_CONTROL_SIZE.y+CONTROL_SPACING.y));
-      map.put(controller.name(), p);
+      map.put(controller.getName(), p);
       row++;
       if (p.y + (DEFAULT_CONTROL_SIZE.y*2) >= panel.getOutline().getHeight())
       {
@@ -597,9 +781,9 @@ Map<String, PVector> buildControlPositionsForPanel(Panel panel)
       }
     }
   }
-
   return map;
 }
+
 Map<String, PVector> buildControlSizesForPanel(Panel panel)
 {
   //println("Building control sizes for panel " + panel.getName());
@@ -609,16 +793,21 @@ Map<String, PVector> buildControlSizesForPanel(Panel panel)
   int row = 0;
   for (Controller controller : panel.getControls())
   {
-    if (controller.name().startsWith("minitoggle_"))
+    if (controller.getName().startsWith("minitoggle_"))
     {
       PVector s = new PVector(DEFAULT_CONTROL_SIZE.y, DEFAULT_CONTROL_SIZE.y);
-      map.put(controller.name(), s);
+      map.put(controller.getName(), s);
+    }
+    else if (controller.getName().startsWith("dropdown_"))
+    {
+      PVector s = new PVector(DEFAULT_CONTROL_SIZE.x, DEFAULT_CONTROL_SIZE.y * 4);
+      map.put(controller.getName(), s);
     }
     else
     {
       PVector s = new PVector(DEFAULT_CONTROL_SIZE.x, DEFAULT_CONTROL_SIZE.y);
-      map.put(controller.name(), s);
-      //println("Added size of " + controller.name() + " to panel. " + s);
+      map.put(controller.getName(), s);
+      //println("Added size of " + controller.getName() + " to panel. " + s);
     }
   }
 
@@ -674,27 +863,36 @@ List<String> getControlNamesForInputPanel()
   controlNames.add(MODE_CHANGE_GRID_SIZE);
   controlNames.add(MODE_CHANGE_SAMPLE_AREA);
   controlNames.add(MODE_CHOOSE_CHROMA_KEY_COLOUR);
+  controlNames.add(MODE_CHANGE_INVERT_MASK);
   controlNames.add(MODE_CHANGE_PIXEL_SCALING);
-  
+
   controlNames.add(MODE_RENDER_PIXEL_DIALOG);
-  controlNames.add(MODE_DRAW_GRID);
+//  controlNames.add(MODE_DRAW_GRID);
   controlNames.add(MODE_DRAW_OUTLINE_BOX);
-  controlNames.add(MODE_DRAW_OUTLINE_BOX_ROWS);
-  controlNames.add(MODE_DRAW_SHADE_BOX_ROWS_PIXELS);
+//  controlNames.add(MODE_DRAW_OUTLINE_BOX_ROWS);
+//  controlNames.add(MODE_DRAW_SHADE_BOX_ROWS_PIXELS);
 
   controlNames.add(MODE_LOAD_VECTOR_FILE);
   controlNames.add(MODE_RESIZE_VECTOR);
   controlNames.add(MODE_MOVE_VECTOR);
-  controlNames.add(MODE_CHANGE_MIN_VECTOR_LINE_LENGTH);
-  //controlNames.add(MODE_VECTOR_PATH_LENGTH_HIGHPASS_CUTOFF);
+  controlNames.add(MODE_ADJUST_PREVIEW_CORD_OFFSET);
+
   controlNames.add(MODE_RENDER_VECTORS);
+  controlNames.add(MODE_CHANGE_MIN_VECTOR_LINE_LENGTH);
+  controlNames.add(MODE_CHANGE_POLYGONIZER);
+  controlNames.add(MODE_CHANGE_POLYGONIZER_LENGTH);
+//  controlNames.add(MODE_CHANGE_POLYGONIZER_ADAPTATIVE_ANGLE);
 
   controlNames.add(MODE_SHOW_IMAGE);
   controlNames.add(MODE_SHOW_VECTOR);
   controlNames.add(MODE_SHOW_QUEUE_PREVIEW);
-  controlNames.add(MODE_SHOW_DENSITY_PREVIEW);
   controlNames.add(MODE_SHOW_GUIDES);
 
+  controlNames.add(MODE_SHOW_DENSITY_PREVIEW);
+  controlNames.add(MODE_PREVIEW_PIXEL_DENSITY_RANGE);
+  controlNames.add(MODE_CHANGE_DENSITY_PREVIEW_POSTERIZE);
+  controlNames.add(MODE_CYCLE_DENSITY_PREVIEW_STYLE);  
+
   
   return controlNames;
 }
@@ -781,7 +979,10 @@ List<String> getControlNamesForDetailPanel()
   controlNames.add(MODE_CHANGE_MACHINE_ACCELERATION);
   controlNames.add(MODE_SEND_MACHINE_SPEED);
   controlNames.add(MODE_SEND_MACHINE_SPEED_PERSIST);
-
+  
+  controlNames.add(MODE_SEND_BUTTON_ACTIVATE);
+  controlNames.add(MODE_SEND_BUTTON_DEACTIVATE);
+  
   controlNames.add(MODE_CHANGE_SERIAL_PORT);
 
   return controlNames;
@@ -822,7 +1023,7 @@ Map<String, String> buildControlLabels()
   result.put(MODE_BEGIN, "Reset queue");
   result.put(MODE_INPUT_BOX_TOP_LEFT, "Select Area");
   result.put(MODE_INPUT_BOX_BOT_RIGHT, "Select BotRight");
-  result.put(MODE_DRAW_OUTLINE_BOX, "Draw Outline box");
+  result.put(MODE_DRAW_OUTLINE_BOX, "Draw Outline selected");
   result.put(MODE_DRAW_OUTLINE_BOX_ROWS, "Draw Outline rows");
   result.put(MODE_DRAW_SHADE_BOX_ROWS_PIXELS, "Draw Outline pixels");
   result.put(MODE_DRAW_TO_POSITION, "Move pen to point");
@@ -918,6 +1119,7 @@ Map<String, String> buildControlLabels()
   result.put(MODE_MOVE_VECTOR, "Move vector");
   result.put(MODE_RENDER_PIXEL_DIALOG, "Render pixels...");
   result.put(MODE_CHOOSE_CHROMA_KEY_COLOUR, "Choose mask colour");
+  result.put(MODE_CHANGE_INVERT_MASK, "Mask mode");
   result.put(MODE_CHANGE_PIXEL_SCALING, "Pixel scaling");
   
   result.put(MODE_PEN_LIFT_UP, "Pen lift");
@@ -954,6 +1156,18 @@ Map<String, String> buildControlLabels()
   result.put(MODE_FLIP_WEBCAM_INPUT, "Flip video");
   result.put(MODE_ROTATE_WEBCAM_INPUT, "Rotate webcam");
 
+  result.put(MODE_SEND_BUTTON_ACTIVATE, "Activate button");
+  result.put(MODE_SEND_BUTTON_DEACTIVATE, "Deactivate button");
+  
+  result.put(MODE_ADJUST_PREVIEW_CORD_OFFSET, "Cord offset");
+  result.put(MODE_CYCLE_DENSITY_PREVIEW_STYLE, "Density preview style");
+  
+  result.put(MODE_CHANGE_DENSITY_PREVIEW_POSTERIZE, "Pixel posterize");
+  result.put(MODE_PREVIEW_PIXEL_DENSITY_RANGE, "Show density range");
+  
+  result.put(MODE_CHANGE_POLYGONIZER, "Polygonizer style");
+  result.put(MODE_CHANGE_POLYGONIZER_LENGTH, "Polygonizer length");
+  result.put(MODE_CHANGE_POLYGONIZER_ADAPTATIVE_ANGLE, "Polygonizer angle");
   
 
   return result;
@@ -1061,6 +1275,7 @@ Set<String> buildControlNames()
   result.add(MODE_CHANGE_MIN_VECTOR_LINE_LENGTH);
   
   result.add(MODE_CHOOSE_CHROMA_KEY_COLOUR);
+  result.add(MODE_CHANGE_INVERT_MASK);
   result.add(MODE_CHANGE_PIXEL_SCALING);
   result.add(MODE_PEN_LIFT_UP);
   result.add(MODE_PEN_LIFT_DOWN);
@@ -1073,13 +1288,13 @@ Set<String> buildControlNames()
   result.add(MODE_SELECT_ROVE_IMAGE_SOURCE);
   result.add(MODE_SEND_START_TEXT);
   result.add(MODE_SHOW_WRITING_DIALOG);
-  result.add(MODE_START_SWIRLING);
-  result.add(MODE_STOP_SWIRLING);
-  result.add(MODE_START_MARKING);
-  result.add(MODE_STOP_MARKING);
-  result.add(MODE_START_SPRITE);
-  result.add(MODE_START_RANDOM_SPRITES);
-  result.add(MODE_STOP_RANDOM_SPRITES);
+//  result.add(MODE_START_SWIRLING);
+//  result.add(MODE_STOP_SWIRLING);
+//  result.add(MODE_START_MARKING);
+//  result.add(MODE_STOP_MARKING);
+//  result.add(MODE_START_SPRITE);
+//  result.add(MODE_START_RANDOM_SPRITES);
+//  result.add(MODE_STOP_RANDOM_SPRITES);
   result.add(MODE_DRAW_NORWEGIAN_DIALOG);
   
   result.add(MODE_LIVE_BLUR_VALUE);
@@ -1094,7 +1309,19 @@ Set<String> buildControlNames()
   result.add(MODE_SHOW_WEBCAM_RAW_VIDEO);
   result.add(MODE_FLIP_WEBCAM_INPUT);
   result.add(MODE_ROTATE_WEBCAM_INPUT);
+  
+  result.add(MODE_SEND_BUTTON_ACTIVATE);
+  result.add(MODE_SEND_BUTTON_DEACTIVATE);
 
+  result.add(MODE_ADJUST_PREVIEW_CORD_OFFSET);
+  result.add(MODE_CYCLE_DENSITY_PREVIEW_STYLE);
+  
+  result.add(MODE_CHANGE_DENSITY_PREVIEW_POSTERIZE);
+  result.add(MODE_PREVIEW_PIXEL_DENSITY_RANGE);
+  
+  result.add(MODE_CHANGE_POLYGONIZER_LENGTH);
+  result.add(MODE_CHANGE_POLYGONIZER);
+  result.add(MODE_CHANGE_POLYGONIZER_ADAPTATIVE_ANGLE);
   
   return result;
 }

drawing.pde

@@ -1,35 +1,35 @@
 /**
- Polargraph controller
- Copyright Sandy Noble 2012.
- 
- This file is part of Polargraph Controller.
- 
- Polargraph Controller is free software: you can redistribute it and/or modify
- it under the terms of the GNU General Public License as published by
- the Free Software Foundation, either version 3 of the License, or
- (at your option) any later version.
- 
- Polargraph Controller is distributed in the hope that it will be useful,
- but WITHOUT ANY WARRANTY; without even the implied warranty of
- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- GNU General Public License for more details.
- 
- You should have received a copy of the GNU General Public License
- along with Polargraph Controller.  If not, see <http://www.gnu.org/licenses/>.
- 
- Requires the excellent ControlP5 GUI library available from http://www.sojamo.de/libraries/controlP5/.
- Requires the excellent Geomerative library available from http://www.ricardmarxer.com/geomerative/.
- 
- This is an application for controlling a polargraph machine, communicating using ASCII command language over a serial link.
- 
- sandy.noble@gmail.com
- http://www.polargraph.co.uk/
- http://code.google.com/p/polargraph/
- */
+  Polargraph controller
+  Copyright Sandy Noble 2015.
+
+  This file is part of Polargraph Controller.
+
+  Polargraph Controller is free software: you can redistribute it and/or modify
+  it under the terms of the GNU General Public License as published by
+  the Free Software Foundation, either version 3 of the License, or
+  (at your option) any later version.
+
+  Polargraph Controller is distributed in the hope that it will be useful,
+  but WITHOUT ANY WARRANTY; without even the implied warranty of
+  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+  GNU General Public License for more details.
+
+  You should have received a copy of the GNU General Public License
+  along with Polargraph Controller.  If not, see <http://www.gnu.org/licenses/>.
+    
+  Requires the excellent ControlP5 GUI library available from http://www.sojamo.de/libraries/controlP5/.
+  Requires the excellent Geomerative library available from http://www.ricardmarxer.com/geomerative/.
+  
+  This is an application for controlling a polargraph machine, communicating using ASCII command language over a serial link.
+
+  sandy.noble@gmail.com
+  http://www.polargraph.co.uk/
+  https://github.com/euphy/polargraphcontroller
+*/
 static final String CMD_CHANGELENGTH = "C01,";
-static final String CMD_CHANGEPENWIDTH = "C02,";
-static final String CMD_CHANGEMOTORSPEED = "C03,";
-static final String CMD_CHANGEMOTORACCEL = "C04,";
+static final String CMD_SETPENWIDTH = "C02,";
+//static final String CMD_CHANGEMOTORSPEED = "C03,";
+//static final String CMD_CHANGEMOTORACCEL = "C04,";
 static final String CMD_DRAWPIXEL = "C05,";
 static final String CMD_DRAWSCRIBBLEPIXEL = "C06,";
 static final String CMD_DRAWRECT = "C07,";
@@ -73,6 +73,9 @@ static final String CMD_SETPENLIFTRANGE = "C45,";
 static final String CMD_SELECT_ROVE_SOURCE_IMAGE = "C46";
 static final String CMD_RENDER_ROVE = "C47";
 
+static final String CMD_ACTIVATE_MACHINE_BUTTON = "C49";
+static final String CMD_DEACTIVATE_MACHINE_BUTTON = "C50";
+
 static final int PATH_SORT_NONE = 0;
 static final int PATH_SORT_MOST_POINTS_FIRST = 1;
 static final int PATH_SORT_GREATEST_AREA_FIRST = 2;
@@ -117,9 +120,7 @@ void sendRequestMachineSize()
 void sendMachineSpec()
 {
   // ask for input to get the new machine size
-  String command = CMD_CHANGEMACHINENAME+newMachineName+",END";
-  addToCommandQueue(command);
-  command = CMD_CHANGEMACHINESIZE+getDisplayMachine().inMM(getDisplayMachine().getWidth())+","+getDisplayMachine().inMM(getDisplayMachine().getHeight())+",END";
+  String command = CMD_CHANGEMACHINESIZE+getDisplayMachine().inMM(getDisplayMachine().getWidth())+","+getDisplayMachine().inMM(getDisplayMachine().getHeight())+",END";
   addToCommandQueue(command);
   command = CMD_CHANGEMACHINEMMPERREV+int(getDisplayMachine().getMMPerRev())+",END";
   addToCommandQueue(command);
@@ -196,13 +197,13 @@ void sendTestPenWidth()
   StringBuilder sb = new StringBuilder();
   sb.append(testPenWidthCommand)
     .append(int(gridSize))
-      .append(",")
-        .append(df.format(testPenWidthStartSize))
-          .append(",")
-            .append(df.format(testPenWidthEndSize))
-              .append(",")
-                .append(df.format(testPenWidthIncrementSize))
-                  .append(",END");
+    .append(",")
+    .append(df.format(testPenWidthStartSize))
+    .append(",")
+    .append(df.format(testPenWidthEndSize))
+    .append(",")
+    .append(df.format(testPenWidthIncrementSize))
+    .append(",END");
   addToCommandQueue(sb.toString());
 }
 
@@ -681,7 +682,6 @@ void sendVectorShapes(RShape vec, float scaling, PVector position, int pathSorti
     if (pointPaths[i] != null) 
     {
       boolean firstPointFound = false;
-
       if (pointPaths[i].length > pathLengthHighPassCutoff)
       {
         List<PVector> filteredPoints = filterPoints(pointPaths[i], VECTOR_FILTER_LOW_PASS, minimumVectorLineLength, scaling, position);
@@ -698,7 +698,7 @@ void sendVectorShapes(RShape vec, float scaling, PVector position, int pathSorti
           if (liftToGetToNewPoint)
             addToCommandQueue(CMD_PENUP+"END");
           // move to this point and put the pen down
-          command = CMD_CHANGELENGTHDIRECT+(int)p.x+","+(int)p.y+","+getMaxSegmentLength()+",END";
+          command = CMD_CHANGELENGTHDIRECT+Math.round(p.x)+","+Math.round(p.y)+","+getMaxSegmentLength()+",END";
           addToCommandQueue(command);
           if (liftToGetToNewPoint)
             addToCommandQueue(CMD_PENDOWN+"END");
@@ -709,7 +709,7 @@ void sendVectorShapes(RShape vec, float scaling, PVector position, int pathSorti
           for (int j=1; j<filteredPoints.size(); j++)
           {
             p = filteredPoints.get(j);
-            command = CMD_CHANGELENGTHDIRECT+(int)p.x+","+(int)p.y+","+getMaxSegmentLength()+",END";
+            command = CMD_CHANGELENGTHDIRECT+Math.round(p.x)+","+Math.round(p.y)+","+getMaxSegmentLength()+",END";
             addToCommandQueue(command);
           }
           lastPoint = new PVector(p.x, p.y);
@@ -717,6 +717,7 @@ void sendVectorShapes(RShape vec, float scaling, PVector position, int pathSorti
       }
     }
   }
+  addToCommandQueue(CMD_PENUP+"END");
   println("finished.");
 }
 
@@ -886,6 +887,7 @@ List<PVector> filterPointsLowPass(RPoint[] points, long filterParam, float scali
 
   // scale and convert all the points first
   List<PVector> scaled = new ArrayList<PVector>(points.length);
+  println("a filterPointsLowPass: Scaled length: " + points.length);
   for (int j = 0; j<points.length; j++)
   {
     RPoint firstPoint = points[j];
@@ -893,14 +895,15 @@ List<PVector> filterPointsLowPass(RPoint[] points, long filterParam, float scali
     p = PVector.mult(p, scaling);
     p = PVector.add(p, position);
     p = getDisplayMachine().inSteps(p);
-    if (getDisplayMachine().getPage().surrounds(p))
+    if (getDisplayMachine().getPictureFrame().surrounds(p))
     {
       p = getDisplayMachine().asNativeCoords(p);
       scaled.add(p);
     }
   }
 
-  if (scaled.size() > 1)
+  println("b filterPointsLowPass: Scaled length: " + scaled.size());
+  if (scaled.size() > 1.0)
   {
     PVector p = scaled.get(0);
     result.add(p);
@@ -909,17 +912,18 @@ List<PVector> filterPointsLowPass(RPoint[] points, long filterParam, float scali
     {
       p = scaled.get(j);
       // and even then, only bother drawing if it's a move of over "x" steps
-      int diffx = int(p.x) - int(result.get(result.size()-1).x);
-      int diffy = int(p.y) - int(result.get(result.size()-1).y);
+      int diffx = abs(int(p.x) - int(result.get(result.size()-1).x));
+      int diffy = abs(int(p.y) - int(result.get(result.size()-1).y));
 
-      if (abs(diffx) > filterParam || abs(diffy) > filterParam)
+      if (diffx > filterParam || diffy > filterParam)
       {
-        //println("Adding point " + p + ", last: " + result.get(result.size()-1));
+        println(j + ". Adding point " + p + " because diffx (" + diffx + ") or diffy (" + diffy + ") is > " + filterParam + ", last: " + result.get(result.size()-1));
         result.add(p);
       }
     }
   }
 
+  println("c filterPointsLowPass: Scaled length: " + result.size());
   if (result.size() < 2)
     result.clear();
 
@@ -940,7 +944,7 @@ void sendMachineStoreMode()
 }
 void sendMachineLiveMode()
 {
-  addToRealtimeCommandQueue(CMD_MACHINE_MODE_LIVE+"END");
+  addToCommandQueue(CMD_MACHINE_MODE_LIVE+"END");
 }
 void sendMachineExecMode()
 {
@@ -964,5 +968,3 @@ void sendDrawRandomSprite(String spriteFilename)
 {
   addToCommandQueue(CMD_DRAW_RANDOM_SPRITE+","+spriteFilename+",100,500,END");
 }
-
-

gamepad.pde

@@ -1,234 +0,0 @@
-//ControllIO controllIO;
-//ControllDevice joypad;
-//
-//ControllButton buttonA;
-//ControllButton buttonB;
-//ControllButton buttonX;
-//ControllButton buttonY;
-//ControllButton buttonL;
-//ControllButton buttonR;
-//ControllButton buttonStart;
-//
-//ControllCoolieHat dpad;
-//
-//List<String> devices = new ArrayList<String>(
-//  Arrays.asList("Controller (Xbox 360 Wireless Receiver for Windows)", 
-//                "Controller (XBOX 360 For Windows)"));
-//
-//String signalFromGamepad = null;
-//
-//static final String BUTTON_A_RELEASED = "ButtonAReleased";
-//static final String BUTTON_B_RELEASED = "ButtonBReleased";
-//static final String BUTTON_L_RELEASED = "ButtonLReleased";
-//static final String BUTTON_R_RELEASED = "ButtonRReleased";
-//static final String BUTTON_START_RELEASED = "ButtonStartReleased";
-//
-//void gamepad_init()
-//{
-//  controllIO = ControllIO.getInstance(this);
-//
-//  try
-//  {
-//    controllIO.printDevices();
-//    for (int i = 0; i<devices.size(); i++)
-//    {
-//      try
-//      {
-//        println("trying " + i + ": " + devices.get(i));
-//        joypad = controllIO.getDevice(devices.get(i));
-//        break;
-//      }
-//      catch (RuntimeException e)
-//      {
-//        println("Requested device (" + devices.get(i) + ") not found.");
-//        joypad = null;
-//      }   
-//    }
-//    
-//    if (joypad != null)
-//    {
-//      joypad.printButtons();
-//    
-//      buttonA = joypad.getButton("Button 0");
-//      buttonB = joypad.getButton("Button 1");
-//      buttonX = joypad.getButton("Button 2");
-//      buttonY = joypad.getButton("Button 3");
-//      
-//      buttonL = joypad.getButton("Button 4");
-//      buttonR = joypad.getButton("Button 5");
-//      
-//      buttonStart = joypad.getButton("Button 7");
-//      
-//      buttonA.plug(this, "buttonARelease", ControllIO.ON_RELEASE);
-//      buttonB.plug(this, "buttonBRelease", ControllIO.ON_RELEASE);
-//      buttonX.plug(this, "buttonXPress", ControllIO.ON_PRESS);
-//      buttonX.plug(this, "buttonXRelease", ControllIO.ON_RELEASE);
-//      buttonY.plug(this, "buttonYRelease", ControllIO.ON_RELEASE);
-//      
-//      buttonL.plug(this, "buttonLRelease", ControllIO.ON_RELEASE);
-//      buttonR.plug(this, "buttonRRelease", ControllIO.ON_RELEASE);
-//      
-//      buttonStart.plug(this, "buttonStartRelease", ControllIO.ON_RELEASE);
-//      
-//      dpad = joypad.getCoolieHat(10);
-//      dpad.setMultiplier(4);
-//      dpad.plug(this, "dpadPress", ControllIO.ON_PRESS);
-//    }
-//  }
-//  catch (RuntimeException e)
-//  {
-//    println("Exception occurred while initialising gamepad: " + e.getMessage());
-//  }
-//}
-//
-//public void buttonARelease()
-//{
-//  signalFromGamepad = BUTTON_A_RELEASED;
-//}
-//public void buttonBRelease()
-//{
-//  signalFromGamepad = BUTTON_B_RELEASED;
-//}
-//public void buttonLRelease()
-//{
-//  signalFromGamepad = BUTTON_L_RELEASED;
-//}
-//public void buttonRRelease()
-//{
-//  signalFromGamepad = BUTTON_R_RELEASED;
-//}
-//public void buttonStartRelease()
-//{
-//  signalFromGamepad = BUTTON_START_RELEASED;
-//}
-//
-//void buttonXPress()
-//{
-//  drawingLiveVideo = true;
-//}
-//void buttonXRelease()
-//{
-//  drawingLiveVideo = false;
-//}
-//void buttonYRelease()
-//{
-//  flipWebcamImage = !flipWebcamImage;
-//}
-//
-//void dpadPress(float x, float y)
-//{
-//  float val = dpad.getValue();
-//  if (val == 6.0)
-//  {
-//    liveSimplification--;
-//    if (liveSimplification < LIVE_SIMPLIFICATION_MIN)
-//      liveSimplification = LIVE_SIMPLIFICATION_MIN;
-//  }
-//  else if (val == 2.0)
-//  {
-//    liveSimplification++;
-//    if (liveSimplification > LIVE_SIMPLIFICATION_MAX)
-//      liveSimplification = LIVE_SIMPLIFICATION_MAX;
-//  }
-//  if (val == 8.0) // left
-//  {
-//    pathLengthHighPassCutoff--;
-//    if (pathLengthHighPassCutoff < PATH_LENGTH_HIGHPASS_CUTOFF_MIN)
-//      pathLengthHighPassCutoff = PATH_LENGTH_HIGHPASS_CUTOFF_MIN;
-//  }
-//  else if (val == 4.0) // right
-//  {
-//    pathLengthHighPassCutoff++;
-//    if (pathLengthHighPassCutoff > PATH_LENGTH_HIGHPASS_CUTOFF_MAX)
-//      pathLengthHighPassCutoff = PATH_LENGTH_HIGHPASS_CUTOFF_MAX;
-//  }
-//
-//  Numberbox n = (Numberbox) getAllControls().get(MODE_LIVE_SIMPLIFICATION_VALUE);
-//  n.setValue(liveSimplification);
-//  n.update();
-//
-//  n = (Numberbox) getAllControls().get(MODE_VECTOR_PATH_LENGTH_HIGHPASS_CUTOFF);
-//  n.setValue(pathLengthHighPassCutoff);
-//  n.update();
-//
-//}
-//
-//void processGamepadInput()
-//{
-//  if (signalFromGamepad != null)
-//  {
-//    println("Signal from gamepad:  " + signalFromGamepad);
-//    if (signalFromGamepad == BUTTON_A_RELEASED)
-//    {
-//      if (captureShape == null && !confirmedDraw)
-//        button_mode_liveCaptureFromLive(); 
-//      else if (captureShape != null && !confirmedDraw)
-//        button_mode_liveClearCapture();
-//      else if (captureShape != null && confirmedDraw)
-//      {
-//        button_mode_liveClearCapture();
-//        button_mode_clearQueue();
-//        confirmedDraw = false;
-//      }
-//    }
-//    else if (signalFromGamepad == BUTTON_B_RELEASED)
-//    {
-//      if (captureShape != null && !confirmedDraw)
-//        button_mode_liveConfirmDraw();
-//    }
-//    else if (signalFromGamepad == BUTTON_L_RELEASED)
-//    {
-//      commandQueueRunning = !commandQueueRunning;
-//    }
-//    else if (signalFromGamepad == BUTTON_R_RELEASED)
-//    {
-//    }
-//    else if (signalFromGamepad == BUTTON_START_RELEASED)
-//    {
-//      preLoadCommandQueue();
-//      button_mode_setPositionHome();
-//    }
-//    
-//      
-//    // clear the signal  
-//    signalFromGamepad = null;
-//  }
-//  
-//}
-
-//void displayGamepadOverlay()
-//{
-//  textSize(40);
-//  fill(255);
-//  
-//  if (captureShape == null)
-//  {
-//    image(aButtonImage, width-400, height-180, 128, 128);
-//    text("SNAP!", width-400, height-200);
-//
-//    textSize(30);
-//    image(dpadYImage, width-600, height-180, 128, 128);
-//    text("Simplify", width-600, height-200);
-//
-//    image(dpadXImage, width-600, height-400, 128, 128);
-//    text("Filter short paths", width-600, height-420);
-//  }
-//  else
-//  {
-//    if (confirmedDraw)
-//    {
-//      image(aButtonImage, width-400, height-180, 128, 128);
-//      text("CANCEL!", width-385, height-200);
-//    }
-//    else
-//    {
-//      image(aButtonImage, width-400, height-180, 128, 128);
-//      text("BACK", width-400, height-200);
-//      image(bButtonImage, width-190, height-180, 128, 128);
-//      text("DRAW!", width-180, height-200);
-//    }
-//  }
-//  
-//  
-//  textSize(12);
-//}

queue.pde

@@ -0,0 +1,7 @@
+/**
+Tools for dealing with a full command queue.
+
+Optimise queue.
+*/
+
+

tabSetup.pde

@@ -1,6 +1,6 @@
 /**
   Polargraph controller
-  Copyright Sandy Noble 2012.
+  Copyright Sandy Noble 2015.
 
   This file is part of Polargraph Controller.
 
@@ -24,7 +24,7 @@
 
   sandy.noble@gmail.com
   http://www.polargraph.co.uk/
-  http://code.google.com/p/polargraph/
+  https://github.com/euphy/polargraphcontroller
 */
 
 Set<Panel> getPanelsForTab(String tabName)
@@ -83,25 +83,35 @@ List<String> buildTabNames()
 
 void initTabs()
 {
-  cp5.tab(TAB_NAME_INPUT).setLabel(TAB_LABEL_INPUT);
-  cp5.tab(TAB_NAME_INPUT).activateEvent(true);
-  cp5.tab(TAB_NAME_INPUT).setId(1);
+  int tabWidth = (int)DEFAULT_CONTROL_SIZE.x;
+  int tabHeight = (int)DEFAULT_CONTROL_SIZE.y;
+  
+  Tab.padding = 13; // that's a weird thing to do
+  
+  Tab input = cp5.getTab(TAB_NAME_INPUT); 
+  input.setLabel(TAB_LABEL_INPUT);
+  input.activateEvent(true);
+  input.setId(1);
 
-  cp5.tab(TAB_NAME_DETAILS).setLabel(TAB_LABEL_DETAILS);
-  cp5.tab(TAB_NAME_DETAILS).activateEvent(true);
-  cp5.tab(TAB_NAME_DETAILS).setId(2);
+  Tab details = cp5.getTab(TAB_NAME_DETAILS); 
+  details.setLabel(TAB_LABEL_DETAILS);
+  details.activateEvent(true);
+  details.setId(2);
+  
+  Tab roving = cp5.getTab(TAB_NAME_ROVING);
+  roving.setLabel(TAB_LABEL_ROVING);
+  roving.activateEvent(true);
+  roving.setId(3);
 
-  cp5.tab(TAB_NAME_ROVING).setLabel(TAB_LABEL_ROVING);
-  cp5.tab(TAB_NAME_ROVING).activateEvent(true);
-  cp5.tab(TAB_NAME_ROVING).setId(3);
+  Tab trace = cp5.getTab(TAB_NAME_TRACE);
+  trace.setLabel(TAB_LABEL_TRACE);
+  trace.activateEvent(true);
+  trace.setId(4);
 
-  cp5.tab(TAB_NAME_TRACE).setLabel(TAB_LABEL_TRACE);
-  cp5.tab(TAB_NAME_TRACE).activateEvent(true);
-  cp5.tab(TAB_NAME_TRACE).setId(4);
-
-  cp5.tab(TAB_NAME_QUEUE).setLabel(TAB_LABEL_QUEUE);
-  cp5.tab(TAB_NAME_QUEUE).activateEvent(true);
-  cp5.tab(TAB_NAME_QUEUE).setId(5);
+  Tab queue = cp5.getTab(TAB_NAME_QUEUE);
+  queue.setLabel(TAB_LABEL_QUEUE);
+  queue.activateEvent(true);
+  queue.setId(5);
 }
 
 public Set<String> buildPanelNames()