![]() |
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: Artist Slideshow addon (with skin and addon integration) - pkscout - 2019-10-27 (2019-10-27, 01:08)Karellen Wrote: Hello @pkscout AS does look for artwork in the extrafanart folder (and has since before I started maintaining it). That seemed to be the unofficial standard for a long time before Kodi 18 shipped so I left it to maintain compatibility with other addons that download artwork. Updating AS for the new artwork structure has ended up not being trivial (mostly because I'm going to have to have a routine to move and rename every piece of existing artwork AS has downloaded), and I've been waiting to make sure the new artwork structure stabilized, so it's going to be awhile. Honestly every time I read up on the new structure it seems overly complicated and requires lots of manual work to get it activated, so my motivation to do the major rewrite wanes. But I'll see if I can't get myself excited about it and do the updates. I'll also include updates so those folders aren't created if all the download options are turned off. That isn't a use case I thought about before. RE: Artist Slideshow addon (with skin and addon integration) - black_eagle - 2019-10-27 There is a bug in your branch 'new-version' Line 73 - if self.LOGDEBUG or (loglevel != xmbc.DEBUG and loglevel != xbmc.INFO): Also I got errors regarding kodi-six with regard to xbmc.LOGDEBUG. I fixed that by importing xbmc instead of from kodi_six import xbmc . Can't remember what the exact error was but I can probably reproduce it by turning debug logging on and off in the addon.
RE: Artist Slideshow addon (with skin and addon integration) - MB1968 - 2019-10-27 (2019-10-27, 19:09)pkscout Wrote: I'll also include updates so those folders aren't created if all the download options are turned off. That isn't a use case I thought about before. Thanks a lot. It will save me time to delete empty redundant folders. It will also speed up browsing when searching for a specific artist. RE: Artist Slideshow addon (with skin and addon integration) - Karellen - 2019-10-27 (2019-10-27, 19:09)pkscout Wrote: AS does look for artwork in the extrafanart folder (and has since before I started maintaining it). That seemed to be the unofficial standard for a long time before Kodi 18 shipped so I left it to maintain compatibility with other addons that download artwork.Ok, thanks for the explanation and understand the extra work involved. ![]() RE: Artist Slideshow addon (with skin and addon integration) - pkscout - 2019-10-28 (2019-10-27, 22:20)Karellen Wrote: Ok, thanks for the explanation and understand the extra work involved. No worries. It kind of comes with the territory when you maintain an addon. Not, you know, the amount of work it takes to maintain Kodi, but still... ![]() RE: Artist Slideshow addon (with skin and addon integration) - pkscout - 2019-10-28 (2019-10-27, 19:40)black_eagle Wrote: There is a bug in your branch 'new-version'A debug log would be helpful, as I run my development environment in debug mode basically constantly and haven't ever seen a crash because of the logging. If using the "standard" xbmc import works, I'll forward it on the kodi-six developer. I need to use kodi-six to get Python 2/3 compatibility in preparation for Kodi Matrix, so we need to get it figured out if the issue is in kodi-six. RE: Artist Slideshow addon (with skin and addon integration) - black_eagle - 2019-10-28 I downloaded a zip of your 'new-version' branch and over-wrote the original files with it. 1st python error.
Easy enough to fix that as it's just a typo. However, when I changed it to if self.LOGDEBUG or (loglevel != xbmc.DEBUG and loglevel != xbmc.INFO): and restarted the slideshow, it threw this error.
Full log https://paste.kodi.tv/homuvazone.kodi ( line 1919) RE: Artist Slideshow addon (with skin and addon integration) - pkscout - 2019-10-29 @black_eagle thanks for that. I have no idea when I changed that line in the new version, as the current version has the correct line. And apparently if you have debug logging turned on in the addon the bad part of that line was never evaluated, so no crash. Anyway, I've uploaded an update to the new-version branch with the fix. There's also some new logic to delete empty artist information and artist image folders, just FYI. The issue was that I had the wrong constant name. Line 73 should read:
RE: [RELEASE] Artist Slideshow 1.3.0 addon (with skin and addon integration) - [email protected] - 2019-10-31 (2013-12-06, 19:03)cw-kid Wrote:Hi..(2013-12-06, 16:32)devilshura Wrote: For Various Artists that has tag "Various Artists" in artist-name.I make local-artist folder for it, and point for not found/no tag-artist with fallback. Just chiming in here but I use SQL to host music on two different PI's and I had some issues with programs being unable to locate things fast enough using SMB://(name)/path so I pointed every to SMB://(ipAddress)/path in kodi and this stopped a lot of reverse look ups for name to ip. Just a thought. My collection has over 2000 artists in it and Artwork has taken months to get found and corrected.. I do like this fallback trick will be checking this out as soon as I find out why my slide show has found some random artists to lock on to for Various Artists..Which happenes to be Dolly Pardon..LOL James RE: Artist Slideshow addon (with skin and addon integration) - Asiapentti - 2019-10-31 @pkscout thank you for adding new Kodi music artwork location to artist slideshow. Only problem i've found, is that now it picks up all images from artist folder (clearlogo, thumb, folder, clearart, landscape). Can you exclude those other artwork types, so that only fanart-fanartxx would be fetched to slideshow. RE: Artist Slideshow addon (with skin and addon integration) - pkscout - 2019-11-01 (2019-10-31, 22:48)Asiapentti Wrote: @pkscout thank you for adding new Kodi music artwork location to artist slideshow. Only problem i've found, is that now it picks up all images from artist folder (clearlogo, thumb, folder, clearart, landscape). Can you exclude those other artwork types, so that only fanart-fanartxx would be fetched to slideshow. $!#& That is going to introduce another entire set of complexities. Artist Slideshow tells the skin where a *folder* of images is, not individual files. So the only way to work around that is to copy all the fanartxxx images to another folder temporarily and then point to that. Or, even worse, temporarily move all the non fanart images somewhere else while that folder is played. I'm going to have to think about this, but right now I'm leaning towards not supporting the Kodi music artwork location. It was never designed with addons like AS in mind and I feel like I'm constantly kludging stuff to get it to work. Maybe someday if Kodi let's a skin display a list of images rather than a folder. RE: Artist Slideshow addon (with skin and addon integration) - Karellen - 2019-11-01 Maybe @DaveBlake has some suggestions? RE: Artist Slideshow addon (with skin and addon integration) - meowmoo - 2019-11-01 I think you could do a resolution check? If the picture is smaller than 1080, don't use it RE: Artist Slideshow addon (with skin and addon integration) - pkscout - 2019-11-02 (2019-11-01, 23:39)meowmoo Wrote: I think you could do a resolution check? The problem isn't figuring out which images to use. That is easy. The problem is that AS has to give the skin a FOLDER reference of images to show, and the skin will then show EVERY IMAGE in that folder. So yes, I can figure out which images are fanart, but then they have to be copied to another folder, potentially over the network. And if the song has multiple artists I have to put all the images from multiple artists in one folder, and rename then all (because Fanart1 will be a duplicate name). Oh and the folder name probably has to be unique to avoid caching problems. And no image can show until that's all done, so there will be a delay if there's an artist with lots of images. And then at the end everything has to be cleaned up. It's possible, sure. What I'm trying to figure out is if it's worth the time to support a feature that is fundamentally not compatible with the way AS has worked for, well, ever. RE: Artist Slideshow addon (with skin and addon integration) - DaveBlake - 2019-11-05 (2019-11-01, 03:54)pkscout Wrote: I'm going to have to think about this, but right now I'm leaning towards not supporting the Kodi music artwork location. It was never designed with addons like AS in mind and I feel like I'm constantly kludging stuff to get it to work.It is true, the new handling of artist artwork in v18 onwards didn't consider the Artist Slideshow at all, or any other addon come to that. I only use one addon myself (the Audio Profile switcher), so am totally oblivious to addon needs unless authors talk to me. I write about the design changes I'm making in the music support subforum, and welcome input, I don't see how else I can be more open and communicative. (2019-11-01, 03:54)pkscout Wrote: Maybe someday if Kodi let's a skin display a list of images rather than a folder.I like the idea of adding a new image control that allows for a "family" of art types e.gf fanart1, fanart2... but have to admit that the GUI engine is a steep learning curve of old code and no one about to give hints, so I may never get to create such a thing let alone for v19. So meanwhile is there some other more simple core change that could help? |