Release CU LRC Lyrics - Printable Version +- Kodi Community Forum (https://forum.kodi.tv) +-- Forum: Support (https://forum.kodi.tv/forumdisplay.php?fid=33) +--- Forum: Add-on Support (https://forum.kodi.tv/forumdisplay.php?fid=27) +---- Forum: Lyrics Add-ons (https://forum.kodi.tv/forumdisplay.php?fid=146) +---- Thread: Release CU LRC Lyrics (/showthread.php?tid=147340) |
RE: CU LRC Lyrics - manfeed - 2016-10-13 (2016-10-10, 21:38)ronie Wrote: that's indeed gonna be hard to clean up. (2016-10-11, 20:58)scott967 Wrote: Note that it's extremely common in K-Pop to have both English and Korean words in lyrics. I have been thinking about this topic and not sure if this approach could work: - Take one line of lyrics (A) - Strip Korean/Chinese characters (B) - Compare (A) to (B). If they are the same go ahead with that line. If different then discard that line with timestamp and everything, since that line isn't needed and go to the next. Excuse mi ignorance if I'm saying a nonsense, since I don't know python nor the logic of the addon... Thanks RE: CU LRC Lyrics - ronie - 2016-10-13 many lrc lyrics i've seen that have korean text, have them on the same line (instead of using two lines) Quote:[00:06.69][ti:Holiday] according to your logics, every line in the song will then be discarded ;-) RE: CU LRC Lyrics - manfeed - 2016-10-13 Yep, I wasn't aware there were of that kind... Something I have learned through my tests is that there is so many types of lyrics (and of varying quality) that is impossible to get always a smooth experience... We will have to live with it... Thanks! RE: CU LRC Lyrics - Snape - 2016-10-13 Hi I'm using the Krypton Beta 2. I rarely use it for music, but today I was playing a song by YES called SURVIVAL. The lyrics that showed were not correct, and were actually kind of rude it showed SURVIVAL by the artist LLOYD BANKS. I'm not sure what to do, would like to contact the developers ideally. Seems like a bug. RE: CU LRC Lyrics - ronie - 2016-10-13 the lyrics are retrieved from various lyrics websites. if they return an incorrect result, there's often not much the addon can do to prevent that. RE: CU LRC Lyrics - manfeed - 2016-10-15 Finally I got something to show... ...more information in: 293864 (thread) Thanks ronie! RE: CU LRC Lyrics - ronie - 2016-10-15 (2016-10-15, 00:31)manfeed Wrote: Thanks ronie! my pleasure. thanx for all the 'in progress' videos, they all bring a big smile to my face :-) now all that's left for me is to backport all those changes to cu lrc for jarvis, i guess... RE: CU LRC Lyrics - manfeed - 2016-10-15 (2016-10-15, 16:07)ronie Wrote:(2016-10-15, 00:31)manfeed Wrote: Thanks ronie! Thanks! The backport would be great! I can't thank you enough for your backing in this project! RE: CU LRC Lyrics - ronie - 2016-10-15 could you test the cu lrc jarvis branch? RE: CU LRC Lyrics - manfeed - 2016-10-15 Tested and working flawlessly! Thank you very much! RE: CU LRC Lyrics - ronie - 2016-10-15 cool, submitted cu lrc v4.2.0 to the jarvis repo. RE: CU LRC Lyrics - manfeed - 2016-10-17 Hi ronie, I don't give up; still fighting the 'evil' korean/chinese characters... I'm testing a new way with variables, substituting direct references to Container(110).ListItem.Property(part... for variables as follows: Code: <variable name="esuno"> This doesn't affect the normal songs and works quite well with lyrics that have the english lyrics line before the korean one like this one... Code: [ti:Last Friday Night ] ...but not the other way round (korean before the english). To get it right I would need some way to know whether a line is english or not, or if has had any korean character removed or not; or whatever way to tell apart one to another that could be used in a variable condition. Or maybe you know another way... Thanks, and sorry for being of the stubborn kind... RE: CU LRC Lyrics - ronie - 2016-10-18 (2016-10-17, 17:44)manfeed Wrote: sorry for being of the stubborn kind... wish you told me in advance, before i decided to get into all of this ;-P i've made another attempt to handle this in the addon, so you can drop that skin code. please checkout my github repo again and let me know. RE: CU LRC Lyrics - manfeed - 2016-10-18 Just brilliant! You did it again, thank you! Only a little issue, I have tested the Jarvis branch and the korean/english lyrics song work ok, but when it finishes the next song get stuck in the first line even it is lrc. Next songs work well again. Tested several times and with other visualizations screens and the same happens. The strange thing is that I also tested the master branch too in Jarvis, copying directly in addons folder and it works flawlessly... RE: CU LRC Lyrics - manfeed - 2016-10-19 (2016-10-18, 17:54)manfeed Wrote: Just brilliant! You did it again, thank you! @ronie Sorry for having pointed to the wrong direction I have made extensive testing and now I know that the problem was not with the song with the characters stripped, but with the next one... by chance that was this one: Code: [ti:Pearl] With the jarvis branch, you can play any song before this one and the result is the same: when starting, it gets stuck in the line "She is a pyramid" (maybe the problem is with the strange preceding lines?). However, with the master branch, there's not such problem, even though I'm in Jarvis... I have tested this with several skins and the behaviour is the same... I hope this helps you to pinpoint the issue with the jarvis branch... Many, many thanks! By the way, I think the reason for my being stubborn is that I started messing with computers with this one... ...if I hadn't been that stubborn I would have given up long ago... |