Chnages suggested by coderabbit

Author: PayalLakra

.github/workflows/release.yaml

@@ -110,7 +110,7 @@ jobs:
           verbose: true  # For better debugging
 
   github_release:
-    needs: [setup_and_build, pypi_publish]
+    needs: [details, setup_and_build, pypi_publish]
     runs-on: ubuntu-latest
     permissions:
       contents: write
@@ -125,7 +125,7 @@ jobs:
           path: dist/
 
       - name: Create Release
-        uses: softprops/action-gh-release@v1
+        uses: softprops/action-gh-release@v2
         with:
           tag_name: ${{ needs.details.outputs.tag_name }}
           files: |

chordspy/app.py

@@ -22,7 +22,6 @@
 from pathlib import Path                                    # For handling file paths in a platform-independent way
 import os                                                   # For file and directory operations
 import webbrowser                                           # For opening the web interface in a browser
-import logging                                              # For logging errors and information
 
 console_queue = queue.Queue()            # Global queue for console messages to be displayed in the web interface
 app = Flask(__name__)                    # Initialize Flask application

chordspy/chords_wifi.py

@@ -8,7 +8,6 @@
 import sys
 import websocket
 import socket
-from scipy.signal import butter, filtfilt
 
 class Chords_WIFI:
     """

chordspy/connection.py

@@ -41,7 +41,7 @@ class Connection:
     - Data streaming to LSL
     - Data recording to CSV files
     - Connection state management
-    - Sample validation and rate monitoring
+    - Sample validation and rate monitorin
     The class maintains separate connection handlers for each protocol (USB/WiFi/BLE)
     and manages their lifecycle. It implements thread-safe operations for concurrent
     data handling and provides clean shutdown procedures.
@@ -94,7 +94,7 @@ def __init__(self):
         self.ble_samples_received = 0              # Count of BLE-specific samples
 
     async def get_ble_device(self):
-        """c
+        """
         Scan for and select a BLE device interactively.
         This asynchronous method: Scans for available BLE devices using Chords_BLE scanner, presents discovered devices to user, handles user selection, returns selected device object.
         Returns:
@@ -235,7 +235,7 @@ def update_sample_rate(self):
             self.sample_count = 0
             self.last_timestamp = now
 
-    def lsl_rate_checker(self, duration=1.0):    # For USB Only(Need modification)
+    def lsl_rate_checker(self, duration=1.0):
         """
         Independently verifies the actual streaming rate of the LSL outlet.
         This method: Collects timestamps over a measurement period -> calculates rate from timestamp differences.

chordspy/static/script.js

@@ -13,7 +13,7 @@ function logError(error) {
             headers: { 'Content-Type': 'application/json' },
             body: JSON.stringify({ error: errorMessage })
         }).catch(() => {
-            console.error('Failed to log error:', errorMessage);
+            consoled.error('Failed to log error:', errorMessage);
         });
 
         console.error(errorMessage);