![]() |
|
[RELEASE] CU Lyrics (Script) Addon - Printable Version +- XBMC Community Forum (http://forum.xbmc.org) +-- Forum: Help and Support (/forumdisplay.php?fid=33) +--- Forum: Add-ons Help and Support (/forumdisplay.php?fid=27) +---- Forum: Lyrics Add-ons (/forumdisplay.php?fid=146) +---- Thread: [RELEASE] CU Lyrics (Script) Addon (/showthread.php?tid=62853) |
- paco - 2009-12-08 03:33 Amet Wrote:try version 0.7.1 on SVN repo installer. I would appreciate the feedback as my music library is almost nonexistent. Hi Zeljko, Thanks for the update. Good job on the (') issue. From the quick test I gave the file, most of my problem files worked! Very nice! Even quotes (") in songs were showing up correctly! This version seems to have trouble when switching songs. Sometimes it refreshes, other times it doesn't. I have to start and stop the script sometimes. No such issue with 0.70. I find that if the next song is available on the HDD, it refreshes. If it has to go to the internet, it doesn't. Great progress with this script! Keep up the nice work! - amet - 2009-12-08 09:04 paco Wrote:Hi Zeljko, Thanks for letting me know, mind trying 0.7.2 on SVN? I have to stop doing this at 3AM, always make silly little mistakes. ![]() Zeljko - paco - 2009-12-08 22:47 Amet Wrote:Thanks for letting me know, mind trying 0.7.2 on SVN? Wow! Awesome job Zeljko! Now when your script fails it makes me check the lyricwiki. Believe it or not, in a few cases I had a misnamed song and in a few cases they had it wrong (always check original album when in doubt!). There is still one issue I have found. Artists with an ampersand in the name (see below): NOTICE: Song: Is This Love NOTICE: Artist: Bob Marley & The Wailers NOTICE: Title:[Bob Marley & The Wailers:Is This Love] NOTICE: http://lyricwiki.org/index.php?title=Bob%20Marley%20%26amp%3B%20The%20Wailers%3AIs%20This%20Love&action=edit It seems that your script sends 2 ampersands when formatting the artist or title. That small problem aside, your script kicks major butt! Great work! - amet - 2009-12-08 23:03 paco Wrote:Wow! Awesome job Zeljko! Now when your script fails it makes me check the lyricwiki. Believe it or not, in a few cases I had a misnamed song and in a few cases they had it wrong (always check original album when in doubt!). Thanks for all the reports, I think its fixed in 0.7.3. Give it a go when you have some time. Zeljko - mvdaat - 2009-12-09 00:57 Hi Amet, Still have the problem with internet radio that it doesn't refresh if I exit the script and go back then it's finding the correct lyric With this new version a lot of songs are working that didn't work before Thanks!!! - paco - 2009-12-09 01:24 Amet Wrote:Thanks for all the reports, I think its fixed in 0.7.3. Give it a go when you have some time. Hello Zeljko, boy are you making it hard to find bugs! This new version is ALMOST perfect. Below is the only songs I've issues with. The funny thing is your script tries to scrape from the proper page but some songs work, yet others come back 'No Lyrics Have Been Found'. Maybe you have some debugging options to see what is causing the script to come back empty. Work Fine: NOTICE: Title:[Creedence Clearwater Revival:Fortunate Son] NOTICE: http://lyricwiki.org/index.php?title=Creedence%20Clearwater%20Revival%3AFortunate%20Son&action=edit NOTICE: Title:[Creedence Clearwater Revival: Down On The Corner] (I had to add space between : & D to avoid smiley) NOTICE: http://lyricwiki.org/index.php?title=Creedence%20Clearwater%20Revival%3ADown%20On%20The%20Corner&action=edit No Lyrics Have Been Found: NOTICE: Title:[Creedence Clearwater Revival:Hey Tonight] NOTICE: http://lyricwiki.org/index.php?title=Creedence%20Clearwater%20Revival%3AHey%20Tonight&action=edit NOTICE: Title:[Creedence Clearwater Revival:Proud Mary] NOTICE: http://lyricwiki.org/index.php?title=Creedence%20Clearwater%20Revival%3AProud%20Mary&action=edit Good luck troubleshooting! - amet - 2009-12-09 11:31 mvdaat Wrote:Hi Amet, Internet radio? wouldnt even know how to test. post the debug log let me have a look. Zeljko - Nuka1195 - 2009-12-09 11:56 internet radio may not trigger the player start event on song change even though the title changes. so it wouldn't work without a constant thread checking song title (not good) - amet - 2009-12-09 12:04 paco Wrote:Hello Zeljko, boy are you making it hard to find bugs! This new version is ALMOST perfect. Below is the only songs I've issues with. The funny thing is your script tries to scrape from the proper page but some songs work, yet others come back 'No Lyrics Have Been Found'. Maybe you have some debugging options to see what is causing the script to come back empty. Hopefully we'll get to the point that it just works . Thanks a lot for your help debugging it.paco Wrote:No Lyrics Have Been Found: hehe, the problem is actually on the LyricWiki page. I have fixed the Creedence Clearwater Revival:Hey Tonight and left Creedence Clearwater Revival:Proud Mary for you to see. the lyrics need to be wrapped in "<lyrics>" tag, whoever edited the Proud Marry and Hey Tonight songs has put "<lyric>" as a tag and thats why we are not seeing it. If a lot of those start popping up I will add the check for "<lyric>" tag. Zeljko - paco - 2009-12-09 16:35 Amet Wrote:hehe, the problem is actually on the LyricWiki page. Good find. I don't know how I missed that when comparing the working songs with the failed songs. I ran into a few other problem songs last night and after reading your post I went back to check them. Yup, the same issue (<lyric> instead of <lyrics>). Here is a sample: http://lyrics.wikia.com/index.php?title=ABBA:Take_A_Chance_On_Me&action=edit http://lyrics.wikia.com/index.php?title=ABBA:Super_Trouper&action=edit I could just edit the songs on lyricwiki, but there may be many more we're not aware of. If it's possible, it might be better to add a check for <lyric> as well (maybe if <lyrics> returns 0 results?). Another suggestion I wanted to make was the handling of Instrumentals. Rather than have the script search every time one is played and return 'No Lyrics Have Been Found', I think it would be better to save the text file for the song with words "Instrumental" for lyrics. That would prevent a needless search (and subsequent fail) when part of a playlist. I notice most Instrumentals have the following tag inside the lyrics {{Instrumental}}. |