Merge branch 'main' into python-recorder

Author: Andreu Gimenez Bolinches

localhost-vao-recorder.py

@@ -3,22 +3,23 @@
 
 import socket
 
-from flask import Flask
+from flask import Flask, request
 
 app = Flask(__name__)
 
 app.secret_key = __name__
 
+# Troubleshooting:
+# https://stackoverflow.com/questions/62705271/connect-to-flask-server-from-other-devices-on-same-network
+
 # ! Workaround: Global variable because it is not JSON serializable and can't
 # ! be stored in Flask.session.
 recorder = Recorder(Config.from_yaml(), setup_name=False)
-print(yaml_dump(recorder.config))
-
 
 @app.route('/setup', methods=['GET'])
 def setup():
-    # TODO get recording name from request
-    recorder.setup()
+    name = request.args.get('name', None)
+    recorder.setup(name=name)
     return str(recorder.next_output.name)
 
 

recorder/device/realsense.py

@@ -10,6 +10,7 @@ def __init__(self, *args, **kwargs):
         if not (streams := self.config.get('streams', [])):
             raise AttributeError('No `streams` found in device')
         assert 'serial_number' in self.config
+        sn = self.config['serial_number']
         self.rsconfig = rs.config()
         self.rsconfig.enable_device(sn)
         for stream in streams.values():