livepilot 1.9.15 → 1.9.16

package/remote_script/LivePilot/clips.py

@@ -50,6 +50,11 @@ def create_clip(song, params):
         raise ValueError("Clip length must be > 0")
 
     clip_slot = get_clip_slot(song, track_index, clip_index)
+    if clip_slot.has_clip:
+        raise ValueError(
+            "Clip slot %d on track %d already has a clip. "
+            "Delete it first with delete_clip." % (clip_index, track_index)
+        )
     clip_slot.create_clip(length)
     clip = clip_slot.clip
 
@@ -147,12 +152,23 @@ def set_clip_loop(song, params):
     clip_index = int(params["clip_index"])
     clip = get_clip(song, track_index, clip_index)
 
-    # Set end before start to avoid Live's loop_start < loop_end clamping.
-    # Expanding the window first ensures the left edge can move freely.
-    if "end" in params:
-        clip.loop_end = float(params["end"])
-    if "start" in params:
-        clip.loop_start = float(params["start"])
+    # Conditional ordering to avoid Live's loop_start < loop_end clamping.
+    # When expanding the window, set the expanding edge first.
+    # When shrinking, set the contracting edge first.
+    new_end = float(params["end"]) if "end" in params else None
+    new_start = float(params["start"]) if "start" in params else None
+
+    if new_end is not None and new_end > clip.loop_end:
+        # Expanding right — set end first so start can move freely
+        clip.loop_end = new_end
+        if new_start is not None:
+            clip.loop_start = new_start
+    else:
+        # Shrinking or only changing start — set start first
+        if new_start is not None:
+            clip.loop_start = new_start
+        if new_end is not None:
+            clip.loop_end = new_end
     if "enabled" in params:
         clip.looping = bool(params["enabled"])
 

package/remote_script/LivePilot/notes.py

@@ -172,6 +172,12 @@ def modify_notes(song, params):
             note.velocity = float(mod["velocity"])
         if "probability" in mod:
             note.probability = float(mod["probability"])
+        if "mute" in mod:
+            note.mute = bool(mod["mute"])
+        if "velocity_deviation" in mod:
+            note.velocity_deviation = float(mod["velocity_deviation"])
+        if "release_velocity" in mod:
+            note.release_velocity = float(mod["release_velocity"])
         modified_count += 1
 
     # Pass the original NoteVector back — Boost.Python requires the C++ type
@@ -273,7 +279,9 @@ def transpose_notes(song, params):
         clip = get_clip(song, track_index, clip_index)
 
     from_time = float(params.get("from_time", 0.0))
-    time_span = float(params.get("time_span", clip.length))
+    # Default span covers from from_time to end of clip, not the full clip length
+    default_span = max(0.0, clip.length - from_time) if clip.length > 0 else 32768.0
+    time_span = float(params.get("time_span", default_span))
 
     # Get notes — returns C++ NoteVector that must be passed back intact
     all_notes = clip.get_notes_extended(0, 128, from_time, time_span)

package/remote_script/LivePilot/server.py

@@ -162,12 +162,12 @@ class LivePilotServer(object):
                             pass
                         continue
                     self._client_connected = True
-                self._client_thread = threading.Thread(
-                    target=self._run_client_session,
-                    args=(client, addr),
-                )
-                self._client_thread.daemon = True
-                self._client_thread.start()
+                    self._client_thread = threading.Thread(
+                        target=self._run_client_session,
+                        args=(client, addr),
+                    )
+                    self._client_thread.daemon = True
+                    self._client_thread.start()
             except socket.timeout:
                 continue
             except OSError: