Kodi Community Forum
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)

Pages: 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89

RE: [RELEASE] CU LRC Lyrics - merge cu and lrc lyrics scripts - Mizaki - 2013-01-25

Thanks, now I can submit AWXi update Smile

RE: [RELEASE] CU LRC Lyrics - merge cu and lrc lyrics scripts - Bobby Blixberg - 2013-01-30

The script does not work for me., here is the snippet from xbmc.log:

Code:
12:27:15 T:3007047488   ERROR: Control 101 in window 13001 has been asked to focus, but it can't
12:27:15 T:2740923200  NOTICE: Thread CFileCache start, auto delete: false
12:27:15 T:3007047488   ERROR: Control 110 in window 13001 has been asked to focus, but it can't
12:27:19 T:3007047488   ERROR: Control 101 in window 13001 has been asked to focus, but it can't
12:27:19 T:2617596736  NOTICE: Thread CFileCache start, auto delete: false
12:27:19 T:2799692608   ERROR: EXCEPTION: Unknown control type for python
12:27:19 T:2799692608   ERROR: EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--
                                             - NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
                                            Error Type: <type 'exceptions.RuntimeError'>
                                            Error Contents: Unknown control type for python
                                            Traceback (most recent call last):
                                              File "/home/xbmc/.xbmc/addons/script.cu.lrclyrics/resources/lib/gui.py", line 397, in onPlayBackStarted
                                                self.function( 2 )
                                              File "/home/xbmc/.xbmc/addons/script.cu.lrclyrics/resources/lib/gui.py", line 355, in myPlayerChanged
                                                self.show_lyrics(lyrics)
                                              File "/home/xbmc/.xbmc/addons/script.cu.lrclyrics/resources/lib/gui.py", line 265, in show_lyrics
                                                self.show_control( 110 )
                                              File "/home/xbmc/.xbmc/addons/script.cu.lrclyrics/resources/lib/gui.py", line 105, in show_control
                                                self.getControl( 101 ).setVisible( controlId == 100 )
                                            RuntimeError: Unknown control type for python
                                            -->End of Python script error report<--
12:27:21 T:3007047488   ERROR: Control 101 in window 13001 has been asked to focus, but it can't
12:27:23 T:3007047488   ERROR: ClearBlock: Trying to clear non existent block /home/xbmc/.xbmc/addons/script.cu.lrclyrics
12:27:24 T:2732530496  NOTICE: Thread Background Loader start, auto delete: false
12:27:25 T:2799692608   ERROR: Failed to run the gc to clean up after running prior to shutting down the Interpreter /home/xbmc/.xbmc/addons/script.cu.lrclyrics/default.py
12:27:25 T:2799692608 WARNING: The python script "/home/xbmc/.xbmc/addons/script.cu.lrclyrics/default.py" has left several classes in memory that we couldn't clean up. The classes include: Window,WindowXMLDialog,Player
12:27:29 T:2732530496  NOTICE: Thread Background Loader start, auto delete: false
12:27:49 T:2732530496  NOTICE: Previous line repeats 2 times.
12:27:49 T:2732530496  NOTICE: Thread CMusicInfoScanner start, auto delete: false
12:28:09 T:2732530496  NOTICE: Previous line repeats 1 times.


RE: [RELEASE] CU LRC Lyrics - merge cu and lrc lyrics scripts - ronie - 2013-01-30

(2013-01-30, 13:31)Bobby Blixberg Wrote: here is the snippet from xbmc.log:

nice, but where's the full Debug Log?

RE: [RELEASE] CU LRC Lyrics - merge cu and lrc lyrics scripts - Bobby Blixberg - 2013-01-30

Here it is:
http://pastebin.com/VuAD2weD

But this looks like an issue with the skin Aeon Nox, with Confluence everything is fine.

EDIT: There's a fix for this...


RE: [RELEASE] CU LRC Lyrics - merge cu and lrc lyrics scripts - sarakha63 - 2013-02-01

this script is amazing andwas all i was looking for, the advantage of both scripts in once.

All is working ok for except one thing, no lyrics are saved , it creates the artist folder but the lyric file isn't created.

IS it a known issue

RE: [RELEASE] CU LRC Lyrics - merge cu and lrc lyrics scripts - ronie - 2013-02-01

(2013-02-01, 17:56)sarakha63 Wrote: this script is amazing andwas all i was looking for, the advantage of both scripts in once.

All is working ok for except one thing, no lyrics are saved , it creates the artist folder but the lyric file isn't created.

IS it a known issue

thanx for the headsup!

i probably broke something in the last update.
looking into it...

RE: [RELEASE] CU LRC Lyrics - merge cu and lrc lyrics scripts - crni - 2013-02-06

(2013-02-01, 19:34)ronie Wrote:
(2013-02-01, 17:56)sarakha63 Wrote: this script is amazing andwas all i was looking for, the advantage of both scripts in once.

All is working ok for except one thing, no lyrics are saved , it creates the artist folder but the lyric file isn't created.

IS it a known issue

thanx for the headsup!

i probably broke something in the last update.
looking into it...
yes I get the same, also shouldn't it make more sense to have option to save lyrics in Album folder not Artist ?



Two Korean scrapers are added in github - hojel - 2013-02-07

I placed pull request on github.
Contribution is adding two lyrics scrapers based on two Korean sites.
Let me know your request or opinion after review.

RE: [RELEASE] CU LRC Lyrics - merge cu and lrc lyrics scripts - curtis-r - 2013-02-17

Love this script. Easily replaced the existing LRC Lyric add-on. Particularly like it's success in finding lyrics, and showing CU lyrics when it's doesn't have LRC available.

Anyhow, I just noticed that very recently it stopped saving lyrics. I check the path and the switch for saving lyrics, and everything appears to be in order. I don't recall touching the settings for some time. Are lyrics still being saved for others out there?

RE: [RELEASE] CU LRC Lyrics - merge cu and lrc lyrics scripts - ronie - 2013-02-17

(2013-02-17, 20:11)curtis-r Wrote: Love this script. Easily replaced the existing LRC Lyric add-on. Particularly like it's success in finding lyrics, and showing CU lyrics when it's doesn't have LRC available.

Anyhow, I just noticed that very recently it stopped saving lyrics. I check the path and the switch for saving lyrics, and everything appears to be in order. I don't recall touching the settings for some time. Are lyrics still being saved for others out there?

fix is on it's way ;-)

RE: Two Korean scrapers are added in github - taxigps - 2013-02-20

(2013-02-07, 12:57)hojel Wrote: I placed pull request on github.
Contribution is adding two lyrics scrapers based on two Korean sites.
Let me know your request or opinion after review.

v1.0.4 - add Korean scraper(Alsong, GomAudio), credit for hojel

RE: [RELEASE] CU LRC Lyrics - merge cu and lrc lyrics scripts - crni - 2013-02-20

(2013-02-17, 20:35)ronie Wrote:
(2013-02-17, 20:11)curtis-r Wrote: Love this script. Easily replaced the existing LRC Lyric add-on. Particularly like it's success in finding lyrics, and showing CU lyrics when it's doesn't have LRC available.

Anyhow, I just noticed that very recently it stopped saving lyrics. I check the path and the switch for saving lyrics, and everything appears to be in order. I don't recall touching the settings for some time. Are lyrics still being saved for others out there?

fix is on it's way ;-)
with recent update lyrics are saved but not in the artist folder ? ...or perhaps I misunderstood 'save lyrics files in the artist folder' setting ? I'd expect lyrics would be saved in the media location artist folder not in the ...\userdata\addon_data\script.cu.lrclyrics\lyrics artist folder ?

If so would it be possible to have option to save lyrics in the %album% folder with the songs ?

e.g.

../Arctic Monkeys/
  • /Suck it and see (2011)/
    • 09. Piledriver Waltz.mp3
      09. Piledriver Waltz.lrc
      ...etc



RE: [RELEASE] CU LRC Lyrics - merge cu and lrc lyrics scripts - ronie - 2013-02-21

(2013-02-20, 09:51)crni Wrote: with recent update lyrics are saved but not in the artist folder ? ...or perhaps I misunderstood 'save lyrics files in the artist folder' setting ? I'd expect lyrics would be saved in the media location artist folder not in the ...\userdata\addon_data\script.cu.lrclyrics\lyrics artist folder ?

If so would it be possible to have option to save lyrics in the %album% folder with the songs ?

e.g.

../Arctic Monkeys/
  • /Suck it and see (2011)/
    • 09. Piledriver Waltz.mp3
      09. Piledriver Waltz.lrc
      ...etc


we indeed don't save the lyrics where the mp3's are.
the reason for this was explained here: http://forum.xbmc.org/showthread.php?tid=147340&pid=1265117#pid1265117

i don't know if there's any way to work around this limitation,
but don't mind spending some time on it.

currently the settings creates separate artists folders within the addon_data directory.

as for the requests to also take the album name into account,
i like the idea, but again i'm unsure whether this could lead to problems somewhere.. in some cases.

@taxigps: do you think it's ok to spend some time on trying to add all of this?

RE: [RELEASE] CU LRC Lyrics - merge cu and lrc lyrics scripts - taxigps - 2013-02-21

(2013-02-21, 02:36)ronie Wrote:
(2013-02-20, 09:51)crni Wrote: with recent update lyrics are saved but not in the artist folder ? ...or perhaps I misunderstood 'save lyrics files in the artist folder' setting ? I'd expect lyrics would be saved in the media location artist folder not in the ...\userdata\addon_data\script.cu.lrclyrics\lyrics artist folder ?

If so would it be possible to have option to save lyrics in the %album% folder with the songs ?

e.g.

../Arctic Monkeys/
  • /Suck it and see (2011)/
    • 09. Piledriver Waltz.mp3
      09. Piledriver Waltz.lrc
      ...etc


we indeed don't save the lyrics where the mp3's are.
the reason for this was explained here: http://forum.xbmc.org/showthread.php?tid=147340&pid=1265117#pid1265117

i don't know if there's any way to work around this limitation,
but don't mind spending some time on it.

currently the settings creates separate artists folders within the addon_data directory.

as for the requests to also take the album name into account,
i like the idea, but again i'm unsure whether this could lead to problems somewhere.. in some cases.

@taxigps: do you think it's ok to spend some time on trying to add all of this?
I'll try to add a setting to save in song directory.

RE: [RELEASE] CU LRC Lyrics - merge cu and lrc lyrics scripts - crni - 2013-02-21

(2013-02-21, 03:33)taxigps Wrote:
(2013-02-21, 02:36)ronie Wrote:
(2013-02-20, 09:51)crni Wrote: with recent update lyrics are saved but not in the artist folder ? ...or perhaps I misunderstood 'save lyrics files in the artist folder' setting ? I'd expect lyrics would be saved in the media location artist folder not in the ...\userdata\addon_data\script.cu.lrclyrics\lyrics artist folder ?

If so would it be possible to have option to save lyrics in the %album% folder with the songs ?

e.g.

../Arctic Monkeys/
  • /Suck it and see (2011)/
    • 09. Piledriver Waltz.mp3
      09. Piledriver Waltz.lrc
      ...etc


we indeed don't save the lyrics where the mp3's are.
the reason for this was explained here: http://forum.xbmc.org/showthread.php?tid=147340&pid=1265117#pid1265117

i don't know if there's any way to work around this limitation,
but don't mind spending some time on it.

currently the settings creates separate artists folders within the addon_data directory.

as for the requests to also take the album name into account,
i like the idea, but again i'm unsure whether this could lead to problems somewhere.. in some cases.

@taxigps: do you think it's ok to spend some time on trying to add all of this?
I'll try to add a setting to save in song directory.

that would be great tnx