joel/freqsplit
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

bugfix: use parameter mono only if passed explicitly

Browse Source 
Caused the tests/test_separation to fail.
This commit is contained in:
Joel Mathew Thomas
2025-02-25 20:50:30 +05:30
parent 3074084ac1
commit 26367e3497
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/freqsplit/input/file_reader.py
+6 -2
View File
@@ -1,7 +1,7 @@
import os import os
import librosa import librosa
def read_audio(file_path, sr=None, mono=False): def read_audio(file_path, sr=None, mono=None):
""" """
Reads an audio file and returns the audio time series and sampling rate. Reads an audio file and returns the audio time series and sampling rate.
@@ -17,7 +17,11 @@ def read_audio(file_path, sr=None, mono=False):
if not os.path.exists(file_path): if not os.path.exists(file_path):
raise FileNotFoundError(f"File not found: {file_path}") raise FileNotFoundError(f"File not found: {file_path}")
try: try:
audio, sr = librosa.load(file_path, sr=sr, mono=mono) # Load with original sampling rate. librosa_kwargs = {"sr": sr}
if mono is not None: # Only add 'mono' if explicitly provided
librosa_kwargs["mono"] = mono
audio, sr = librosa.load(file_path, **librosa_kwargs)
return audio, sr return audio, sr
except Exception as e: except Exception as e:
raise RuntimeError(f"Error reading the audio file: {e}") raise RuntimeError(f"Error reading the audio file: {e}")