Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lyrics: Polish comments appear in text of lyrics #4228

Closed
mattmartini opened this issue Jan 10, 2022 · 7 comments · Fixed by #4229
Closed

lyrics: Polish comments appear in text of lyrics #4228

mattmartini opened this issue Jan 10, 2022 · 7 comments · Fixed by #4229
Labels
bug bugs that are confirmed and actionable

Comments

@mattmartini
Copy link

When downloading lyrics for songs there is Polish text appearing at the top and bottom of the text. This text is saved into the mp3. It seems like this should be resolved with something like the remove_credits() function.

When lyrics are downloaded the text

Tekst piosenki:

appears at the top of the lyrics. [Translates to "lyrics"]

Additionally, the text

Dodaj interpretację do tego tekstu »

Historia edycji tekstu

appears at the bottom of the text. [Translates to "add interpretation to this text" and "history of the text editing"]

This appears on every lyrics downloaded.

This was mentioned in issues #3904 and #4043 as side issues, but since it was never fixed I am opening this issue in hopes that it will be addressed.

Problem

Running this command in verbose (-vv) mode:

$ beet -vv lyrics -p Papercut
user configuration: /home/<USER>/.config/beets/config.yaml
data directory: /home/<USER>/.config/beets
plugin paths:
ImageMagick version check failed: [Errno 2] No such file or directory: 'magick': 'magick'
artresizer: method is (2, (6, 9, 7), True)
lyrics: Disabling google source: no API key configured.
lastgenre: Loading canonicalization tree /home/<USER>/.local/lib/python3.6/site-packages/beetsplug/lastgenre/genres-tree.yaml
Sending event: pluginload
library database: /media/audio/DBs/beets_musiclibrary.db
library directory: /media/audio/Music_Collection
Sending event: library_opened
lyrics: lyrics already present: AAC 256kbps  2:36  02/06  Rap	JAY‐Z & Linkin Park	2004Collision Course	Big Pimpin’ / Papercut
Tekst piosenki:
Hahaha, wasting your talent Randy!
Yeah, you're ready?

[SNIP - full text of lyrics removed for brevity]

We doin'... big pimpin' up in N.Y.C.
It's just that Jigga Man, Pimp C, and B.U.N.B-B-B
Dodaj interpretację do tego tekstu »

Historia edycji tekstu
Sending event: cli_exit

Led to this problem:

Extraneous lines in lyrics text:

Tekst piosenki:
Dodaj interpretację do tego tekstu »
Historia edycji tekstu

Here's a link to the music files that trigger the bug (if relevant):

It happens with every music file.

Setup

This happens on both Mac and Linux computers:

* OS: Mac OS 10.14.6 (Mojave)
* Python version: 3.9.8
* beets version: 1.6.0
* Turning off plugins made problem go away (yes/no): no, it is a plugin issue
* OS: Linux Ubuntu 18.04.6 LTS (Bionic Beaver)
* Python version: 3.6.9
* beets version: 1.6.0
* Turning off plugins made problem go away (yes/no): no, it is a plugin issue

My configuration (output of beet config) is:

lyrics:
    bing_lang_from: []
    auto: no
    force: no
    bing_client_secret: REDACTED
    bing_lang_to:
    google_API_key: REDACTED
    google_engine_ID: REDACTED
    genius_api_key: REDACTED
    fallback:
    local: no
    sources:
    - google
    - musixmatch
    - genius
    - tekstowo
@mattmartini mattmartini changed the title Polish comments appear in text of lyrics lyrics: Polish comments appear in text of lyrics Jan 10, 2022
@sampsyo sampsyo added the bug bugs that are confirmed and actionable label Jan 10, 2022
@sampsyo
Copy link
Member

sampsyo commented Jan 10, 2022

Thanks for the detailed report; this is super helpful!

@hlubach
Copy link

hlubach commented Mar 31, 2022

beets version 1.6.0
Python version 3.8.10
plugins: convert, discogs, duplicates, embedart, fetchart, ftintitle, info, lastgenre, lyrics, missing, permissions, scrub, zero
Just tried beet lyrics -f yesterday.
But when going through the Library still in some songs the Polish lyrics.

Tekst piosenki:

appears at the top of the lyrics. [Translates to "lyrics"]

Additionally, the text

Dodaj interpretację do tego tekstu »

Historia edycji tekstu

appears at the bottom of the text. [Translates to "add interpretation to this text" and "history of the text editing"]

It seems that this item is still not solved

@sampsyo
Copy link
Member

sampsyo commented Mar 31, 2022

To confirm, are you running the latest source version of beets?

@hlubach
Copy link

hlubach commented Mar 31, 2022 via email

@wisp3rwind
Copy link
Member

That's probably the same thing that makes our weekly integration tests fail, cf. e.g. https://github.com/beetbox/beets/actions/runs/2046155673

The Tekstowo source needs to be fixed; I guess they updated their website structure at some point and our scraper now picks up extraneous text?

@sampsyo
Copy link
Member

sampsyo commented Mar 31, 2022

Indeed; looks like it. Maybe this needs a new issue?

@hlubach
Copy link

hlubach commented Apr 1, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bugs that are confirmed and actionable
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants