adamjakab / BeetsPluginBpmAnalyser

A beets plugin for obsessive-compulsive music geeks to add BPM information to their songs.
MIT License
23 stars 4 forks source link

Not working on import #9

Closed 2V3EvG4LMJFdRe closed 4 months ago

2V3EvG4LMJFdRe commented 1 year ago

Problem

First of all awesome job with the script. It works with beet -vv bpmanalyser whatever and

plugins: fetchart embedart lyrics discogs spotify convert bpmanalyser replaygain
autotag: yes
ignore_hidden: yes
original_date: yes
languages: en
va_name: Compilation
directory: /redacted/
import:
    copy: yes
    write: yes
    move: no
convert:
    auto: yes
    quiet: yes
    delete_originals: no
    format: mp3
    mp3:
      command: ffmpeg -i $source -q:a 0 -map_metadata 0 $dest
      extension: mp3
discogs:
    user_token: redacted
fetchart:
    auto: yes
    sources: fanarttv itunes amazon coverart albumart
    fanarttv_key: redacted
embedart: 
    auto: yes
    maxwidth: 1000
    remove_art_file: yes
lyrics:
    auto: yes
    force: yes
    import.write: yes
    sources: genius
    google_API_key: redacted
replaygain:
  auto: yes
  overwrite: yes
bpmanalyser:
  auto: yes
  dry-run: no
  write: yes
  threads: 2
  force: yes
  quiet: yes
replace:
    'featuring': 'feat\.'
    '’': "'"
    '[“”]': '"'
    '[…]': '...'
    '[-]': '-'
    '[−]': '-'
    '[—]': '-'
    '[Ⅰ]': I
    '[Ⅱ]': II

However, on import, no tag is written to the files automatically.

Setup

adamjakab commented 4 months ago

Thanks for reporting this. I will take a look.

adamjakab commented 4 months ago

I tested this and I confirm that the latest v1.4.0 and above correctly runs bpmanalyser plugin on import when auto is set to yes. @2V3EvG4LMJFdRe - your configuration above is correct and probably in an older version this was not working. I must have fixed this in the past without realizing it.