Fix SACD Imports

beetbox · Nov 23, 2024 · 32e9e58 · 32e9e58
1 parent ef328ed
commit 32e9e58
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 0 deletions.
diff --git a/beets/autotag/match.py b/beets/autotag/match.py
@@ -208,6 +208,10 @@ def track_distance(
     if item.mb_trackid:
         dist.add_expr("track_id", item.mb_trackid != track_info.track_id)
 
+    # Penalize mismatching disc numbers.
+    if track_info.medium and item.disc:
+        dist.add_expr("medium", item.disc != track_info.medium)
+
     # Plugins.
     dist.update(plugins.track_distance(item, track_info))
 

diff --git a/beets/config_default.yaml b/beets/config_default.yaml
@@ -207,6 +207,7 @@ match:
         track_index: 1.0
         track_length: 2.0
         track_id: 5.0
+        medium: 1.0
     preferred:
         countries: []
         media: []

diff --git a/docs/changelog.rst b/docs/changelog.rst
@@ -17,6 +17,9 @@ Bug fixes:
 * Check if running python from the Microsoft Store and provide feedback to install
   from python.org.
   :bug:`5467`
+* Fix bug where matcher doesn't consider medium number when importing. This makes
+  it difficult to import hybrid SACDs and other releases with duplicate tracks.
+  :bug:`5148`
 
 For packagers: