Amet Wrote:Thanks for all the reports, I think its fixed in 0.7.3. Give it a go when you have some time.

Zeljko

mvdaat Wrote: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 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.

paco Wrote: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 Revivalroud Mary]
NOTICE: http://lyricwiki.org/index.php?title=Creedence%20Clearwater%20Revival%3AProud%20Mary&action=edit

Good luck troubleshooting!

Amet Wrote:hehe, the problem is actually on the LyricWiki page.

I have fixed the Creedence Clearwater Revival:Hey Tonight and left Creedence Clearwater Revivalroud 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 Wrote: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?).

paco Wrote: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}}.

Amet Wrote:I will add that as well, have to find one with instrumental. I dont even have music library scanned in XBMC

Thanks
Zeljko

paco Wrote:Sounds great.

Here are links to a few Instrumentals:

http://lyrics.wikia.com/index.php?title=The_Police:Behind_My_Camel&action=edit

http://lyrics.wikia.com/index.php?title=Rush:YYZ&action=edit

http://lyrics.wikia.com/index.php?title=Van_Halen:Spanish_Fly&action=edit

Quote:0.7.4 is available via SVN Repo Installer / 09.12.09
- 0.7.4 Fixed:Search for <lyric> tag as well, save to file if song is instrumental, thanks again paco

Nuka1195 Wrote: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 Wrote:0.7.4 is on SVN Repo, when you get a chance give it a go.

Zeljko

thomash72 Wrote:I just wanted to say that I love this script! Together with Transparency! 2.11 this is a jawdropping combination! Thank you for your hard work on this - I just wanted to let you know that it's highly appreciated.

Best Regards,
Thomas

steve1977 Wrote:I am also using Transparency! 2.11 and unfortunately, dont get it to work. The "old script" XBMC Lyrics finds the song, but then gives an error fetching the lyrics. The "new one" CU Lyrics states "please select a song", but nothing shows up.

Any thoughts?!

paco Wrote:Hello Zeljko, your new version is very good. One suggestion is to not save a txt file when a search comes back empty or when the title is there but there are no lyrics. After testing this version, I had a few files saved that had the following:

<!-- PUT LYRICS HERE (and delete this entire line) -->

IMO it's better to let the user know that no results were found (maybe a message to ask them to check the spelling or capitalization of their files?). I had a couple files that were missing a capital letter in the name. The downside to saving an empty file is that there is no 'refresh' option (I had to manually delete the text file, go to lyricwiki and find out whether it or my tags were at fault and fix the issue).

All in all, this release is excellent and I'm loving how well it works. When you're finished with this version, you may want to consider adding these features to your Batch Lyrics script.

P.S. Instrumentals are working great!

Quote:0.7.5 is available via SVN Repo Installer / 05.01.2010
- 0.7.5 Fixed:do not save to file if lyrics download failed, added auto scroll timer(enable it under script settings)