beets can't fetch lyrics from lrclib.net #5102
Comments
|
Can you please fill out a bit more of the bug report template? It helps us work out what's going on. Specifically:
The full template can be found here: https://github.com/beetbox/beets/blob/master/.github/ISSUE_TEMPLATE/bug-report.md?plain=1 |
jackwilsdon
added
the
needinfo
|
Not OP, but here is my info: name: "\U0001F41B Bug report" ProblemRunning this command in verbose ( $ beet -vv lyrics -f Metallica Loaduser configuration: /home/user/.config/beets/config.yaml Led to this problem: Setup
My configuration (output of lyrics:
bing_lang_from: []
auto: yes
sources: lrclib
synced: yes
bing_client_secret: REDACTED
bing_lang_to:
google_API_key: REDACTED
google_engine_ID: REDACTED
genius_api_key: REDACTED
fallback:
force: no
local: no
plugins:
- chroma
- fromfilename
- acousticbrainz
- embedart
- fetchart
- lastgenre
- lyrics
- mbsync
- metasync
- replaygain
- bucket
- inline
- the
- badfiles
- duplicates
- info
- mbcollection
- mbsubmit
- missing
- random
- types
- alternatives
asciify_paths: yes
format_item: $artist - $album - $track. $title ($length)
format_album: $albumartist - $album
sort_item: albumartist+ album+ disc+ track+
per_disc_numbering: yes
terminal_encoding: utf8
clutter:
- Thumbs.DB
- .DS_Store
- '*.m3u'
- .pls
- '*.jpg'
max_filename_length: 64
id3v23: yes
import:
write: yes
copy: yes
move: no
link: no
hardlink: no
resume: ask
quiet: no
log: ~/tmp/beetslog
detail: yes
musicbrainz:
user: gderber
pass: REDACTED
match:
strong_rec_thresh: 0.08
max_rec:
missing_tracks: low
unmatched_tracks: medium
track_length: strong
preferred:
countries:
- US
- XW
- CA
- XE
- GB|UK
- AU
- DE
media: [CD, Digital Media|File]
paths:
singleton: '%ifdef{sort,$sort,Unsorted}/%ifdef{album_holiday,$album_holiday}%ifdef{soundtrack_type,$soundtrack_type}/%the{%asciify{$albumartist}}/Non-Album/%the{%asciify{$title}}'
comp: '%ifdef{sort,$sort,Unsorted}/%ifdef{album_holiday,$album_holiday}%ifdef{soundtrack_type,$soundtrack_type}/%the{%asciify{$albumartist}}/%the{%asciify{$album}}/%if{$multidisc,%if{$lmultidisc,$disc,%right{$disc,1}}}$track - %asciify{$title}'
default: '%ifdef{sort,$sort,Unsorted}/%ifdef{album_holiday,$album_holiday}%ifdef{soundtrack_type,$soundtrack_type}/%the{%asciify{$albumartist}}/%ifdef{series,%the{$series}}/%ifdef{series_no,$series_no - }%the{%asciify{$album}}/%if{$multidisc,%if{$lmultidisc,$disc,%right{$disc,1}}}$track - %asciify{$title}'
chroma:
auto: yes
acousticbrainz:
auto: yes
force: no
tags: []
edit:
itemfields: track title artist album
albumfields: album albumartist
embedart:
auto: yes
compare_threshold: 0
ifempty: yes
maxwidth: 0
remove_art_file: no
quality: 0
fetchart:
auto: yes
cautious: yes
cover_names: cover front art album folder
minwidth: 480
maxwidth: 0
quality: 0
max_filesize: 0
enforce_ratio: no
sources:
- filesystem
- coverart
- itunes
- amazon
- albumart
google_key: REDACTED
google_engine: 001442825323518660753:hrh5ch1gjzm
fanarttv_key: REDACTED
lastfm_key: REDACTED
store_source: no
high_resolution: no
deinterlace: no
cover_format:
ftintitle:
auto: no
format: feat. {0}
keyfinder:
auto: no
lastgenre:
auto: yes
fallback: ''
source: album
whitelist: ~/.config/beets/genres.txt
min_weight: 10
count: 1
canonical: no
force: yes
separator: ', '
prefer_specific: no
title_case: yes
metasync:
source: amarok
replaygain:
backend: gstreamer
overwrite: yes
targetlevel: 89
auto: yes
threads: 8
parallel_on_import: no
per_disc: no
peak: 'true'
r128: [Opus]
r128_targetlevel: 84
bucket:
bucket_year: [1980-89]
extrapolate: yes
bucket_alpha: []
bucket_alpha_regex: {}
item_fields:
multidisc: 1 if disctotal > 1 else 0
lmultidisc: 1 if disctotal > 9 else 0
the:
a: yes
the: yes
patterns: []
strip: no
format: '{0}, {1}'
thumbnails:
force: yes
badfiles:
commands:
ogg: myoggchecker --opt1 --opt2
flac: flac --test --warnings-as-errors --silent
duplicates:
album: no
checksum: ''
copy: ''
count: no
delete: no
format: ''
full: no
keys: []
merge: no
move: ''
path: no
tiebreak: {}
strict: no
tag: ''
mbcollection:
auto: no
collection: ''
remove: no
mbsubmit:
threshold: medium
format: $track. $title - $artist ($length)
missing:
format: $albumartist - $album - $track - $title
count: no
total: no
album: no
types:
rating: int
playcount: intNot really sure if that helps or not. https://lrclib.net/search/Metallica%20Hero%20of%20the%20Day for example exists. |
|
I also have this issue. |
|
Same here, I get a I can't use other sources because Navidrome only supports synced lyrics, and My lyrics config: Maybe LRCLIB had some API changes under the hood?
|
|
I have fixed this issue locally, will submit a PR in a couple of days! |
|
The PR with a fix has not been submitted.
The issue was with the plugin implementation: see part of the diff from the PR: Previously the plugin attempted to I updated the implementation to perform a Additionally, LRCLib is now prioritised over other sources in the default configuration since the search is quick and reliable, and it provides high quality lyrics. |
Create 'helpers.ConfigMixin' which sets up testing configuration. This is helpful for tests (e.g. test_lyrics.py) that only need the configuration and do not require temp dir. (#5102) Refactor lyrics tests to fix the issue global beets config issue. Additionally, add 'integration_test' mark that can be used to mark tests that should only run once a week.
Create 'helpers.ConfigMixin' which sets up testing configuration. This is helpful for tests (e.g. test_lyrics.py) that only need the configuration and do not require temp dir. (#5102) Refactor lyrics tests to fix the issue global beets config issue. Additionally, add 'integration_test' mark that can be used to mark tests that should only run once a week.
Create 'helpers.ConfigMixin' which sets up testing configuration. This is helpful for tests (e.g. test_lyrics.py) that only need the configuration and do not require temp dir. (#5102) Refactor lyrics tests to fix the issue global beets config issue. Additionally, add 'integration_test' mark that can be used to mark tests that should only run once a week.
Create 'helpers.ConfigMixin' which sets up testing configuration. This is helpful for tests (e.g. test_lyrics.py) that only need the configuration and do not require temp dir. (#5102) Refactor lyrics tests to fix the issue global beets config issue. Additionally, add 'integration_test' mark that can be used to mark tests that should only run once a week.
Create 'helpers.ConfigMixin' which sets up testing configuration. This is helpful for tests (e.g. test_lyrics.py) that only need the configuration and do not require temp dir. (#5102) Refactor lyrics tests to fix the issue global beets config issue. Additionally, add 'integration_test' mark that can be used to mark tests that should only run once a week.
|
same issue. will this get merged? |
Create 'helpers.ConfigMixin' which sets up testing configuration. This is helpful for tests (e.g. test_lyrics.py) that only need the configuration and do not require temp dir. (#5102) Refactor lyrics tests to fix the issue global beets config issue. Additionally, add 'integration_test' mark that can be used to mark tests that should only run once a week.
Create 'helpers.ConfigMixin' which sets up testing configuration. This is helpful for tests (e.g. test_lyrics.py) that only need the configuration and do not require temp dir. (#5102) Refactor lyrics tests to fix the issue global beets config issue. Additionally, add 'integration_test' mark that can be used to mark tests that should only run once a week.
Create 'helpers.ConfigMixin' which sets up testing configuration. This is helpful for tests (e.g. test_lyrics.py) that only need the configuration and do not require temp dir. (beetbox#5102) Refactor lyrics tests to fix the issue global beets config issue. Additionally, add 'integration_test' mark that can be used to mark tests that should only run once a week.
I want to fetch synced lyrics from lrclib but beet can't find them even if i can find them manually on lrclib.
Here's my config:
The text was updated successfully, but these errors were encountered: