ok100 / lyvi

🎶 Command-line lyrics (and more!) viewer
http://ok100.github.com/lyvi/
Do What The F*ck You Want To Public License
172 stars 28 forks source link

Fails to find lyrics fairly often #11

Closed nixnull closed 11 years ago

nixnull commented 11 years ago

I'm using master, specifically the arch aur 'lyvi-git' package.

It fails to find lyrics for some songs that ~mpd~ ncmpcpp succeeds at.

I tried looking at the source code to figure out how lyvi finds the lyrics, but I didn't succeed.

ok100 commented 11 years ago

Hi, please can you test with rewrite branch (see https://bbs.archlinux.org/viewtopic.php?pid=1316595#p1316595)? I'll probably not fix anything in the old version but rewrite branch will be merged in master soon.

sahib commented 11 years ago

Also please make sure to have a recent version of libglyr before trying. Just reinstall it from git. (Post your glyrc --version if problem persists).

P.S: MPD has nothing to do with lyrics-retrieval.

nixnull commented 11 years ago

I've obtained the rewrite branch and installed all dependencies listend on the linked archlinux forum comment. Running ~/git/lyvi/lyvi.py gives me:

Traceback (most recent call last): File "./lyvi.py", line 3, in import lyvi File "/home/nichdel/git/lyvi/lyvi/init.py", line 153, in player = lyvi.players.find() File "/home/nichdel/git/lyvi/lyvi/players/init.py", line 23, in find if lyvi.config['default_player']: TypeError: 'NoneType' object is not subscriptable

nixnull commented 11 years ago

sahib:

I now have 1.0.1 Bloody Boar.

And I meant to say ncmpcpp, not mpd. Apologies.

sahib commented 11 years ago

Until ok100 has figured out that traceback:

Can you try the following and tell me if it gave you the requested output:

glyrc lyrics --artist '<the artist you tried>' --title '<the title you tried>' -v3

If not, please paste this output too.

nixnull commented 11 years ago

It did not find the lyrics for Da Vinci's Notebook's Internet Porn.

[nichdel@manjaro lyvi]$ glyrc lyrics --artist "Da Vinci's Notebook" --title 'Internet Porn'

---- Triggering: musictree ---- Triggering: local ---- Triggering: lyricswiki ---- Triggering: vagalume ---- Triggering: lipwalk ---- Triggering: lyrdb ---- Triggering: elyrics ---- Triggering: lyricstime ---- Triggering: chartlyrics ---- Triggering: lyrix ---- Triggering: lyricsvip ---- Triggering: magistrix ---- Triggering: metallum ---- Triggering: lyricsreg ---- Triggering: metrolyrics

ncmpcpp seems to fetch it from lyricwiki.com, it that helps.

sahib commented 11 years ago

This means that this is libglyr's fault. :-(

Problem:

Solution:

Thanks nichdel for your work.

ok100 commented 11 years ago

@nichdel: The traceback is fixed. I really should learn how to do unit tests in Python. Also, Lyvi can now find lyrics for "Da Vinci's Notebook - Internet Porn".

nixnull commented 11 years ago

@ok100 @sahib

Thanks a lot guys! It's working great now.