From f8db03e725d528569abb13c22c2878c213822023 Mon Sep 17 00:00:00 2001
From: chromhelm <chromhelm@users.noreply.github.com>
Date: Fri, 29 May 2015 16:51:55 +0200
Subject: [PATCH 1/3] - Saving same bytes in HardwareSerial::write

---
 hardware/arduino/avr/cores/arduino/HardwareSerial.cpp | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/hardware/arduino/avr/cores/arduino/HardwareSerial.cpp b/hardware/arduino/avr/cores/arduino/HardwareSerial.cpp
index 4022698934a..a2029a8b0d4 100644
--- a/hardware/arduino/avr/cores/arduino/HardwareSerial.cpp
+++ b/hardware/arduino/avr/cores/arduino/HardwareSerial.cpp
@@ -213,6 +213,7 @@ void HardwareSerial::flush()
 
 size_t HardwareSerial::write(uint8_t c)
 {
+  _written = true;
   // If the buffer and the data register is empty, just write the byte
   // to the data register and be done. This shortcut helps
   // significantly improve the effective datarate at high (>
@@ -220,7 +221,6 @@ size_t HardwareSerial::write(uint8_t c)
   if (_tx_buffer_head == _tx_buffer_tail && bit_is_set(*_ucsra, UDRE0)) {
     *_udr = c;
     sbi(*_ucsra, TXC0);
-     _written = true;
     return 1;
   }
   tx_buffer_index_t i = (_tx_buffer_head + 1) % SERIAL_TX_BUFFER_SIZE;
@@ -244,10 +244,8 @@ size_t HardwareSerial::write(uint8_t c)
   _tx_buffer_head = i;
 	
   sbi(*_ucsrb, UDRIE0);
-  _written = true;
   
   return 1;
 }
 
-
 #endif // whole file

From 219cc6c4e482403705f555992c50c59f1f1eff0f Mon Sep 17 00:00:00 2001
From: Wilhelm <wilhelmwiens@gmail.com>
Date: Thu, 5 Apr 2018 22:15:33 +0200
Subject: [PATCH 2/3] Add clear button to SerialPlotter

---
 app/src/processing/app/SerialPlotter.java | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/app/src/processing/app/SerialPlotter.java b/app/src/processing/app/SerialPlotter.java
index 363753749fe..262b525687d 100644
--- a/app/src/processing/app/SerialPlotter.java
+++ b/app/src/processing/app/SerialPlotter.java
@@ -37,6 +37,7 @@ public class SerialPlotter extends AbstractMonitor {
 
   private final StringBuffer messageBuffer;
   private JComboBox<String> serialRates;
+  private JButton clearButton;
   private Serial serial;
   private int serialRate, xCount;
 
@@ -231,6 +232,8 @@ public SerialPlotter(BoardPort port) {
 
     messageBuffer = new StringBuffer();
     graphs = new ArrayList<>();
+    
+    clearButton.addActionListener(ae -> {graphs.clear();});
   }
 
   protected void onCreateWindow(Container mainPane) {
@@ -248,16 +251,20 @@ protected void onCreateWindow(Container mainPane) {
     for (String serialRateString : serialRateStrings) serialRates.addItem(serialRateString + " " + tr("baud"));
 
     serialRates.setMaximumSize(serialRates.getMinimumSize());
+    
+    clearButton = new JButton(tr("Clear output"));
 
     pane.add(Box.createHorizontalGlue());
     pane.add(Box.createRigidArea(new Dimension(8, 0)));
     pane.add(serialRates);
+    pane.add(clearButton);
 
     mainPane.add(pane, BorderLayout.SOUTH);
   }
 
   protected void onEnableWindow(boolean enable) {
     serialRates.setEnabled(enable);
+    clearButton.setEnabled(enable);
   }
 
   private void onSerialRateChange(ActionListener listener) {

From c76a8f98054856153efdfd216dc47540e46246d6 Mon Sep 17 00:00:00 2001
From: Wilhelm <wilhelmwiens@gmail.com>
Date: Fri, 6 Apr 2018 00:22:11 +0200
Subject: [PATCH 3/3] Added label support to SerialPlotter Supports csv style
 header and colon separated <label>:<value> Two command where added. All
 command start with # CLEAN ->  will clean the scope SIZE:<value> -> set the
 xaxis size. Right now it is limited to 10-5000.

---
 app/src/processing/app/SerialPlotter.java     | 127 +++++++++++++++---
 .../app/helpers/CircularBuffer.java           |  10 +-
 2 files changed, 117 insertions(+), 20 deletions(-)

diff --git a/app/src/processing/app/SerialPlotter.java b/app/src/processing/app/SerialPlotter.java
index 262b525687d..b20c4670691 100644
--- a/app/src/processing/app/SerialPlotter.java
+++ b/app/src/processing/app/SerialPlotter.java
@@ -26,6 +26,9 @@
 import java.util.ArrayList;
 import javax.swing.*;
 import javax.swing.border.EmptyBorder;
+
+import com.jcraft.jsch.Buffer;
+
 import java.awt.*;
 import java.awt.event.ActionListener;
 import java.awt.geom.AffineTransform;
@@ -42,14 +45,18 @@ public class SerialPlotter extends AbstractMonitor {
   private int serialRate, xCount;
 
   private ArrayList<Graph> graphs;
-  private final static int BUFFER_CAPACITY = 500;
+  private int buffer_capacity = 500;
+  private final static int BUFFER_CAPACITY_DEFAULT = 500;
+  private final static int BUFFER_CAPACITY_MAX = 5000;
+  private final static int BUFFER_CAPACITY_MIN = 10;
 
   private static class Graph {
     public CircularBuffer buffer;
+    public String label;
     private Color color;
 
-    public Graph(int id) {
-      buffer = new CircularBuffer(BUFFER_CAPACITY);
+    public Graph(int id, int capacity) {
+      buffer = new CircularBuffer(capacity);
       color = Theme.getColorCycleColor("plotting.graphcolor", id);
     }
 
@@ -147,12 +154,12 @@ public void paintComponent(Graphics g1) {
       }
 
       // handle data count
-      int cnt = xCount - BUFFER_CAPACITY;
-      if (xCount < BUFFER_CAPACITY) cnt = 0;
+      int cnt = xCount - buffer_capacity;
+      if (xCount < buffer_capacity) cnt = 0;
         
       double zeroTick = ticks.getTick(0);
       double lastTick = ticks.getTick(ticks.getTickCount() - 1);
-      double xTickRange = BUFFER_CAPACITY / ticks.getTickCount();
+      double xTickRange = buffer_capacity / ticks.getTickCount();
         
       for (int i = 0; i < ticks.getTickCount() + 1; i++) {
           String s;
@@ -168,7 +175,7 @@ public void paintComponent(Graphics g1) {
               s = String.valueOf((int)(xTickRange * i)+cnt);
               fBounds = fm.getStringBounds(s, g);
               sWidth = (int)fBounds.getWidth()/2;
-              xValue = (int)((bounds.width - xOffset - xPadding) * ((xTickRange * i) / BUFFER_CAPACITY) + xOffset);
+              xValue = (int)((bounds.width - xOffset - xPadding) * ((xTickRange * i) / buffer_capacity) + xOffset);
           }
           // draw graph x axis, ticks and labels
           g.setColor(boundsColor);
@@ -185,13 +192,24 @@ public void paintComponent(Graphics g1) {
       g.drawLine(xOffset, (int) transformY(zeroTick), bounds.width - xPadding, (int)transformY(zeroTick));
         
       g.setTransform(AffineTransform.getTranslateInstance(xOffset, 0));
-      float xstep = (float) (bounds.width - xOffset - xPadding) / (float) BUFFER_CAPACITY;
-      int legendLength = graphs.size() * 10 + (graphs.size() - 1) * 3;
-
+      float xstep = (float) (bounds.width - xOffset - xPadding) / (float) buffer_capacity;
+      // draw legend
+      int legendXOffset = 0;
       for(int i = 0; i < graphs.size(); ++i) {
         graphs.get(i).paint(g, xstep, minY, maxY, rangeY, bounds.height);
-        if(graphs.size() > 1) {    
-          g.fillRect(bounds.width - (xOffset + legendLength + 10) + i * 13, 10, 10, 10);
+        if(graphs.size() > 1) {
+          //draw legend rectangle
+          g.fillRect(10 + legendXOffset, 10, 10, 10);
+          legendXOffset += 13;
+          //draw label
+          g.setColor(boundsColor);
+          String s = graphs.get(i).label;
+          if(s != null && s.length() > 0) {
+			Rectangle2D fBounds = fm.getStringBounds(s, g);
+			int sWidth = (int)fBounds.getWidth();
+			g.drawString(s, 10 + legendXOffset, 10 + (int)fBounds.getHeight() /2);
+			legendXOffset += sWidth + 3;
+	      }
         }
       }
     }
@@ -270,6 +288,17 @@ protected void onEnableWindow(boolean enable) {
   private void onSerialRateChange(ActionListener listener) {
     serialRates.addActionListener(listener);
   }
+  
+  private void setNewBufferCapacity(int capacity){
+    if(buffer_capacity != capacity) {
+      if(capacity > BUFFER_CAPACITY_MAX) capacity = BUFFER_CAPACITY_MAX;
+      else if(capacity < BUFFER_CAPACITY_MIN) capacity = BUFFER_CAPACITY_MIN;
+      buffer_capacity = capacity;
+      for(int i = 0; i < graphs.size(); i++) {
+        graphs.get(i).buffer.newCapacity(capacity);
+      }
+    }
+  }
 
   public void message(final String s) {
     messageBuffer.append(s);
@@ -287,19 +316,81 @@ public void message(final String s) {
       if(parts.length == 0) {
         continue;
       }
-
+	
       int validParts = 0;
+      int validLabels = 0;
       for(int i = 0; i < parts.length; ++i) {
-        try {          
-          double value = Double.valueOf(parts[i]);
+        Double value = null;
+        String label = null;
+        
+        // all commands start with #
+        if(parts[i].startsWith("#") && parts[i].length() > 1) {
+          String command = parts[i].substring(1, parts[i].length()).trim();
+          if(command.equals("CLEAR")) {
+            graphs.clear();
+          }
+          else if(command.startsWith("SIZE:")) {
+            String[] subString = parts[i].split("[:]+");
+            int newSize = BUFFER_CAPACITY_DEFAULT;
+            if(subString.length > 1) {
+              try {
+                newSize = Integer.parseInt(subString[1]);
+              } catch (NumberFormatException e) {
+                // Ignore
+              }
+            }
+            setNewBufferCapacity(newSize);
+          }
+        } else {
+          if(parts[i].contains(":")) {
+            // get label
+            String[] subString = parts[i].split("[:]+");
+            
+            if(subString.length > 0) {
+              int labelLength = subString[0].length();
+              
+              if(labelLength > 32) {
+                labelLength = 32;
+              }
+              label = subString[0].substring(0, labelLength);
+            } else {
+              label = "";
+            }
+            
+            if(subString.length > 1) {
+              parts[i] = subString[1];
+            } else {
+              parts[i] = "";
+            }
+          }
+          try {          
+            value = Double.valueOf(parts[i]);
+          } catch (NumberFormatException e) {
+              // ignored
+          }
+          //CSV header
+          if(label == null && value == null) {
+            label = parts[i];
+          }
+        }
+          
+        if(value != null) {
           if(validParts >= graphs.size()) {
-            graphs.add(new Graph(validParts));
+            graphs.add(new Graph(validParts, buffer_capacity));
           }
           graphs.get(validParts).buffer.add(value);
           validParts++;
-        } catch (NumberFormatException e) {
-          // ignore
+          
+        }
+        if(label != null) {
+          if(validLabels >= graphs.size()) {
+            graphs.add(new Graph(validLabels, buffer_capacity));
+          }
+          graphs.get(validLabels).label = label;
+          validLabels++;
         }
+        if(validParts > validLabels) validLabels = validParts;
+        else if(validLabels > validParts) validParts = validLabels;
       }
     }
 
diff --git a/app/src/processing/app/helpers/CircularBuffer.java b/app/src/processing/app/helpers/CircularBuffer.java
index 8396c8fd2e1..26ce00b95e7 100644
--- a/app/src/processing/app/helpers/CircularBuffer.java
+++ b/app/src/processing/app/helpers/CircularBuffer.java
@@ -4,10 +4,10 @@
 
 public class CircularBuffer {
 
-  private final double[] elements;
+  private double[] elements;
   private int start = -1;
   private int end = -1;
-  private final int capacity;
+  private int capacity;
 
   public void add(double num) {
     end = (end + 1) % capacity;
@@ -36,6 +36,12 @@ public CircularBuffer(int capacity) {
     this.capacity = capacity;
     elements = new double[capacity];
   }
+  
+  public void newCapacity(int capacity) {
+    elements = new double[capacity];
+    this.capacity = capacity;
+    start = end = 0;
+  }
 
   public double min() {
     if (size() == 0) {