Release Artist Slideshow addon (with skin and addon integration) - 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: Program Add-ons (https://forum.kodi.tv/forumdisplay.php?fid=151) +---- Thread: Release Artist Slideshow addon (with skin and addon integration) (/showthread.php?tid=124880) 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
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
|
RE: [RELEASE] Artist Slideshow 1.3.0 addon (with skin and addon integration) - Crowly - 2012-08-20 (2012-08-19, 16:21)pkscuot Wrote: I used some of my google-fu and found something that might fix this and the problem with extended characters in a username. I don't have a good way to test it on my system, but it at least didn't break anything. For folks with these problems, please download and try this branch:I created a new user on my system with user name Øystein, then testet both v 1.41 and 1.4.2.2, it didn't change anything. As far as i can see from the log, the char set is not using a extended ascii table, only the first 128 chars. Where ex. Ø = 216. v1.4.2.2 log: Code: 18:11:37 T:2828 NOTICE: -->Python Interpreter Initialized<-- Log from v1.4.2 version, tested before i installed v1.4.2.2 Code: 18:04:14 T:2988 NOTICE: -->Python Interpreter Initialized<-- RE: [RELEASE] Artist Slideshow 1.3.0 addon (with skin and addon integration) - pkscout - 2012-08-21 (2012-08-20, 11:40)karmark Wrote: Hi Yes. If you go to your XBMC userdata folder (on OSX for instance that is ~/Library/Application Support/XBMC/), it'll be in: userdata/addon_data/script.artistslideshow/ArtistSlideshow If you delete all the folders in that directory that will completely clear the cache. (2012-08-20, 17:52)dreamkp Wrote: thanks for getting on that so fast. Well at least I broke it more than before. That's a change anyway. Quote:(2012-08-19, 16:21)pkscuot Wrote: I don't have a good way to test it on my system, but it at least didn't break anything.that should be easy if you just need some test material I use iTunes for all my music, and when I added a special character to an artist it didn't seem to get into XBMC. I'll probably try creating another music source that is file based and do what you suggest. I'll look at the CDArt Manager stuff. I might even break down and post something in the plugin development forum. RE: [RELEASE] Artist Slideshow 1.3.0 addon (with skin and addon integration) - Oddsodz - 2012-08-21 Funny you mention "Björk". I have a few of her albums and she is tagged with that spelling. XBMC and Artist slideshow has been fine with here. Not had an issue at all ever. Works as intended RE: [RELEASE] Artist Slideshow 1.3.0 addon (with skin and addon integration) - dreamkp - 2012-08-21 (2012-08-21, 17:29)Oddsodz Wrote: Funny you mention "Björk". I have a few of her albums and she is tagged with that spelling. XBMC and Artist slideshow has been fine with here. Not had an issue at all ever. Works as intendedHi Oddsodz, don't get confused. We are talking about saving of information on the local disk. Bio/Fanart, etc. so it doesn't have to be scraped every time. See here That's were we have problems with special chars. During playpack even artists with special chars are displayed fine. Regards, Maikl RE: [RELEASE] Artist Slideshow 1.3.0 addon (with skin and addon integration) - pkscout - 2012-08-21 (2012-08-20, 17:52)dreamkp Wrote: thanks for getting on that so fast. OK, let's try this again. This code works properly on my OSX and Windows machines with Björk as an artist. I'm hoping this also fixes the Windows username issue, as it turns out one of the problems relates to the fact that you can't write to the XBMC log in UTF-8. So I had to make sure to re-encode any text string before writing it out. Here's the updated test code for the non-ascii character issue. If we can get this one resolved, I'll merge it into master and do another pull request into the repo (which will also include the notification fixes then). RE: [RELEASE] Artist Slideshow 1.3.0 addon (with skin and addon integration) - dreamkp - 2012-08-21 (2012-08-21, 19:40)pkscuot Wrote:(2012-08-20, 17:52)dreamkp Wrote: thanks for getting on that so fast. Hi pkscout, it's getting better. Now artistbio.nfo, artistsalbums.nfo and artistsimilar.nfo are written. Then i get a script error and the log below. Seems only artistimageshtbackdrops.nfo, artistimageslastfm.nfo and the *.tbn are missing now. We're close ------------------------- 20:52:44 T:2672 ERROR: Error Type: <type 'exceptions.UnicodeDecodeError'> 20:52:44 T:2672 ERROR: Error Contents: 'ascii' codec can't decode byte 0xc3 in position 146: ordinal not in range(128) 20:52:44 T:2672 ERROR: Traceback (most recent call last): File "C:\xbmc_frodo\portable_data\addons\script.artistslideshow\default.py", line 742, in <module> slideshow = Main() File "C:\xbmc_frodo\portable_data\addons\script.artistslideshow\default.py", line 134, in __init__ self._use_correct_artwork() File "C:\xbmc_frodo\portable_data\addons\script.artistslideshow\default.py", line 182, in _use_correct_artwork self._start_download() File "C:\xbmc_frodo\portable_data\addons\script.artistslideshow\default.py", line 351, in _start_download lastfmlist = self._get_images('lastfm') File "C:\xbmc_frodo\portable_data\addons\script.artistslideshow\default.py", line 562, in _get_images log( 'asking for images from: %s' %self.url ) File "C:\xbmc_frodo\portable_data\addons\script.artistslideshow\default.py", line 77, in log message = 'script.artistslideshow: %s' % txt.encode("utf-8") UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 146: ordinal not in range(128) ------------------------- Regards, Maikl RE: [RELEASE] Artist Slideshow 1.3.0 addon (with skin and addon integration) - pkscout - 2012-08-21 (2012-08-21, 20:57)dreamkp Wrote: it's getting better. Now artistbio.nfo, artistsalbums.nfo and artistsimilar.nfo are written. Then i get a script error and the log below. I posted another patch. Same link as above. Two questions: 1- Do you have AS set to use both local and remote artwork? That's the only way I can figure you got the error where you did. I tested this time with that option. 2- Are you running this with debugging enabled when you first test? The error you got this time was another error where AS was trying to log a non-ascii character to the XBMC log. The only time those logs are written is when XBMC is in debug mode (or at least logging debug messages). It needed to get fixed either way, but I just want to make sure I didn't miss something else. RE: [RELEASE] Artist Slideshow 1.3.0 addon (with skin and addon integration) - dreamkp - 2012-08-21 (2012-08-21, 21:33)pkscuot Wrote:(2012-08-21, 20:57)dreamkp Wrote: it's getting better. Now artistbio.nfo, artistsalbums.nfo and artistsimilar.nfo are written. Then i get a script error and the log below. Hi pkscuot, it works To answer your questions. 1) Yes, i have it set to both local and remote (can't remember why. read the wiki and decided thats the way to go^^) --> my settings.xml 2) xbmc was not set to debugging mode during the previous test runs. but i use it often to iron out errors (advancedsettings.xml is set to <loglevel>0</loglevel> atm) One error did show up in the log, but it didn't stop the script: --------- 22:47:32 T:1392 ERROR: C:\xbmc_frodo\portable_data\addons\script.artistslideshow\default.py:457: UnicodeWarning: Unicode equal comparison failed to convert both arguments to Unicode - interpreting them as being unequal if( xbmc.getInfoLabel( self.ARTISTSLIDESHOW ) == self.BlankDir): --------- Not sure if that can lead to some unknown effects. Thank you very much for your efforts. You did me a great favour. Regards, Maikl RE: [RELEASE] Artist Slideshow 1.3.0 addon (with skin and addon integration) - pkscout - 2012-08-21 (2012-08-21, 23:03)dreamkp Wrote: [quote='pkscuot' pid='1174213' dateline='1345577612'] Yea, that could probably cause some problems in some very specific (and probably not too often used) situations. I believe I've corrected that, and I posted an updated patch in the same place. If you check one more time, that would be great. Then I think I can merge this in and put in another pull request to get the updates into the official repo. RE: [RELEASE] Artist Slideshow 1.3.0 addon (with skin and addon integration) - Oddsodz - 2012-08-22 (2012-08-21, 18:16)dreamkp Wrote: That's were we have problems with special chars. During playpack even artists with special chars are displayed fine. Ah, that will teach me not to read everything over again. My bad. Keep up the good work folks. This is a a very VERY good addon. And one that needs to be added to the core of XBMC at some point. It's is just too good not to have on by default. RE: [RELEASE] Artist Slideshow 1.3.0 addon (with skin and addon integration) - dreamkp - 2012-08-22 (2012-08-21, 23:27)pkscuot Wrote: If you check one more time, that would be great. Then I think I can merge this in and put in another pull request to get the updates into the official repo. Hi pkscuot, new version is still working and no errors or warnings in the log. Well done. Regards, Maikl RE: [RELEASE] Artist Slideshow 1.3.0 addon (with skin and addon integration) - Crowly - 2012-08-22 (2012-08-21, 19:40)pkscuot Wrote: Here's the updated test code for the non-ascii character issue.Did a test and now it works But i had to delete the other (older) versions before it would, in case thats of any importance. RE: [RELEASE] Artist Slideshow 1.3.0 addon (with skin and addon integration) - Acrobat76 - 2012-08-23 I think this may be more a skin issue, but lately I noticed that I am no longer able to see "just" the fullscreen artist slideshow fanart but there is also the progress bar, song name, artists name etc displayed. This happens on both confluence and quartz. Is there any way to remove this and just have the fanart slideshow displaying? RE: [RELEASE] Artist Slideshow 1.3.0 addon (with skin and addon integration) - pkscout - 2012-08-23 (2012-08-23, 18:11)Acrobat76 Wrote: I think this may be more a skin issue, but lately I noticed that I am no longer able to see "just" the fullscreen artist slideshow fanart but there is also the progress bar, song name, artists name etc displayed. This happens on both confluence and quartz. Is there any way to remove this and just have the fanart slideshow displaying? I did create some notifications in the last version, but not like you are describing. The only thing I added was a notification when downloading of images started and then when they are finished. There is a new beta version that turns the notifications off by default, and I'll be pushing those changes into the official repo tomorrow (most likely). So either try the beta now or wait until this weekend and see if the update fixes it. If it doesn't, I'd like to see a few screenshots and a debug log to see if we can figure out what's going on. RE: [RELEASE] Artist Slideshow 1.3.0 addon (with skin and addon integration) - Acrobat76 - 2012-08-23 Thanks Pkscout. Im pretty sure it isnt due to your add on per se. And its not a bug. I guess I am posting in the wrong thread as I think its related to skin(s) or even xbmc in general. I have no idea how to post screenshots from an atv2, but basically what I mean is that whereas a few weeks ago the fullscreen mode used to just give me a slideshow of the artist pics, now I get an overlay with all the song's info (artist, album, album pic, progress bar etc). Id like to get rid of this extra overlay and just have the fullscreen slideshow. Guess I need to post in the general forums or the skin sub forums.... |