From 3e2f8766276584d39a0495ac9ec54f75ca0570b9 Mon Sep 17 00:00:00 2001
From: Joel Collins <gitlab@jtcollins.net>
Date: Mon, 7 Dec 2020 16:49:00 +0000
Subject: [PATCH 1/4] Added mjpeg bitrate to settings

---
 openflexure_microscope/camera/pi.py | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/openflexure_microscope/camera/pi.py b/openflexure_microscope/camera/pi.py
index 9f3bf1cd..0be4e2dc 100644
--- a/openflexure_microscope/camera/pi.py
+++ b/openflexure_microscope/camera/pi.py
@@ -94,6 +94,14 @@ class PiCameraStreamer(BaseCamera):
 
         self.jpeg_quality: int = 100  #: int: JPEG quality
         self.mjpeg_quality: int = 75  #: int: MJPEG quality
+        self.mjpeg_bitrate: int = -1  #: int: MJPEG quality
+        # Solid bitrate options:
+        # -1: Maximum
+        # 25000000: High
+        # 17000000: Normal
+        # 5000000: Low (may impact fast AF)
+        # 2500000: Very low (may impact fast AF)
+
 
         # Start stream recording (and set resolution)
         self.start_stream()
@@ -140,6 +148,7 @@ class PiCameraStreamer(BaseCamera):
             "numpy_resolution": self.numpy_resolution,
             "jpeg_quality": self.jpeg_quality,
             "mjpeg_quality": self.mjpeg_quality,
+            "mjpeg_bitrate": self.mjpeg_bitrate,
             "picamera": {},
         }
 
@@ -409,7 +418,7 @@ class PiCameraStreamer(BaseCamera):
                     self.stream,
                     format="mjpeg",
                     quality=self.mjpeg_quality,
-                    bitrate=-1,  # RWB: disable bitrate control
+                    bitrate=self.mjpeg_bitrate,  # RWB: disable bitrate control
                     # (bitrate control makes JPEG size less good as a focus
                     # metric)
                     splitter_port=1,

From 42e0dfd39cbc13ef305cc3d3ccee8d036d5d4e90 Mon Sep 17 00:00:00 2001
From: Joel Collins <gitlab@jtcollins.net>
Date: Mon, 7 Dec 2020 17:05:56 +0000
Subject: [PATCH 2/4] Replaced MJPEG quality setting with MJPEG bitrate

---
 .../settingsComponents/cameraSettings.vue     | 46 +++++++++++++------
 1 file changed, 32 insertions(+), 14 deletions(-)

diff --git a/openflexure_microscope/api/static/src/components/tabContentComponents/settingsComponents/cameraSettings.vue b/openflexure_microscope/api/static/src/components/tabContentComponents/settingsComponents/cameraSettings.vue
index 9a5aa29f..7e705482 100644
--- a/openflexure_microscope/api/static/src/components/tabContentComponents/settingsComponents/cameraSettings.vue
+++ b/openflexure_microscope/api/static/src/components/tabContentComponents/settingsComponents/cameraSettings.vue
@@ -53,18 +53,19 @@
             <h4>Image quality</h4>
 
             <div class="uk-child-width-1-2" uk-grid>
-              <div v-if="mjpeg_quality !== undefined">
+              <div v-if="mjpeg_bitrate !== undefined">
                 <label class="uk-form-label" for="form-stacked-text"
-                  >Web stream quality (%)</label
+                  >Web stream bitrate</label
                 >
-                <div class="uk-form-controls">
-                  <input
-                    v-model="mjpeg_quality"
-                    class="uk-input uk-form-small"
-                    type="number"
-                    step="1"
-                  />
-                </div>
+                <select v-model="mjpeg_bitrate" class="uk-select uk-form-small">
+                  <option
+                    v-for="option in bitrateOptions"
+                    :key="option.value"
+                    :value="option.value"
+                  >
+                    {{ option.text }}
+                  </option>
+                </select>
               </div>
 
               <div v-if="jpeg_quality !== undefined">
@@ -81,6 +82,16 @@
                 </div>
               </div>
             </div>
+
+            <span
+              v-if="
+                mjpeg_bitrate !== undefined &&
+                  mjpeg_bitrate < 17000000 &&
+                  mjpeg_bitrate != -1
+              "
+              class="uk-text-danger uk-margin-top-small"
+              >This stream bitrate may impact fast autofocus performance</span
+            >
           </div>
 
           <button
@@ -123,8 +134,15 @@ export default {
         analog_gain: undefined,
         digital_gain: undefined
       },
-      mjpeg_quality: undefined,
-      jpeg_quality: undefined
+      mjpeg_bitrate: undefined,
+      jpeg_quality: undefined,
+      bitrateOptions: [
+        { text: "Maximum", value: -1 },
+        { text: "High", value: 25000000 },
+        { text: "Normal", value: 17000000 },
+        { text: "Low", value: 5000000 },
+        { text: "Very low", value: 2500000 }
+      ]
     };
   },
 
@@ -145,7 +163,7 @@ export default {
         .then(response => {
           const cameraSettings = response.data.camera;
           // Get base camera settings
-          this.mjpeg_quality = cameraSettings.mjpeg_quality;
+          this.mjpeg_bitrate = cameraSettings.mjpeg_bitrate;
           this.jpeg_quality = cameraSettings.jpeg_quality;
           // Get Pi Camera settings if they exist
           if (cameraSettings.picamera) {
@@ -164,7 +182,7 @@ export default {
       // make the numbers be strings... TypeScript would solve this...
       var payload = {
         camera: {
-          mjpeg_quality: parseInt(this.mjpeg_quality),
+          mjpeg_bitrate: parseInt(this.mjpeg_bitrate),
           jpeg_quality: parseInt(this.jpeg_quality),
           picamera: {
             shutter_speed: parseFloat(this.picamera.shutter_speed),

From 0626b13bfd29177fe4a6760a6d481c746eba697f Mon Sep 17 00:00:00 2001
From: Joel Collins <gitlab@jtcollins.net>
Date: Mon, 7 Dec 2020 17:08:47 +0000
Subject: [PATCH 3/4] Formatting

---
 openflexure_microscope/camera/pi.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/openflexure_microscope/camera/pi.py b/openflexure_microscope/camera/pi.py
index 0be4e2dc..4f892734 100644
--- a/openflexure_microscope/camera/pi.py
+++ b/openflexure_microscope/camera/pi.py
@@ -102,7 +102,6 @@ class PiCameraStreamer(BaseCamera):
         # 5000000: Low (may impact fast AF)
         # 2500000: Very low (may impact fast AF)
 
-
         # Start stream recording (and set resolution)
         self.start_stream()
         # Wait until frames are available

From 47680fd23b474529dac9609ca5fb5cc537a82f2c Mon Sep 17 00:00:00 2001
From: Joel Collins <gitlab@jtcollins.net>
Date: Mon, 7 Dec 2020 17:11:02 +0000
Subject: [PATCH 4/4] v2.9.1

---
 CHANGELOG.md   | 7 +++++++
 pyproject.toml | 2 +-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index dfa7040f..86210dfa 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,3 +1,10 @@
+# [v2.9.1](https://gitlab.com/openflexure/openflexure-microscope-server/compare/v2.9.0...v2.9.1) (2020-12-07)
+
+
+* Server: Added mjpeg bitrate to settings ([3e2f876](https://gitlab.com/openflexure/openflexure-microscope-server/commit/3e2f876))
+* JS Client: Replaced MJPEG quality setting with MJPEG bitrate ([42e0dfd](https://gitlab.com/openflexure/openflexure-microscope-server/commit/42e0dfd))
+
+
 # [v2.9.0](https://gitlab.com/openflexure/openflexure-microscope-server/compare/v2.8.0...v2.9.0) (2020-12-07)
 
 ## Developer changes
diff --git a/pyproject.toml b/pyproject.toml
index 1f65701a..b90efc9e 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -4,7 +4,7 @@ build-backend = "poetry.masonry.api"
 
 [tool.poetry]
 name = "openflexure-microscope-server"
-version = "2.9.0"
+version = "2.9.1"
 description = "Python module, and Flask-based web API, to run the OpenFlexure Microscope."
 
 authors = [