plympton;541317 Wrote: > There's a few things about the code that I think needs some attention to > make it more robust. First, for each line of text or image, it makes a > separate request, and each of these requests is done in parallel > (assync, right?). I think they can step on each other if there's some > lag, or an error, and that's probably not good. We could easily send > SDT an array/table of request strings and have it process them all at > once. [Edit: I saw some weirdness in the logs when it was fetching > data every second, but after switching to 10 second polling, I haven't > seen that problem generally] > > Might be tricky to do in CustomClock, since it's geared towards reading > in the config data and getting results right away. But you could have > it building up the request array, and then have it go to a function > that: > 1. Checks to see if the data is updated, if not, use cached > information. > 2. Sends the appropriate request, gets results, and parses them for all > strings. > I think it should be doable to handle this in Custom Clock, it's just a matter of collecting all sdttext items before requesting them. If you could implement the JSON command on the plugin side that would be a good start.
We basically just need some code that iterates through all items in self.configItems and finds all items of stdtext and gets their formatting string and create the request to the server. The response would have modify the text in the self.items structure for all corresponding items. Do you think users will have a lot of sdt items on the same screen ? Would the different items have to be synchronized so they switch to new values simultaneously ? If there are just a few items and they doesn't have to be completely synchronized, it might not be worth to combine them into a single request. plympton;541317 Wrote: > > Also, this block of text doesn't process if the player is in play mode. > Is there a reason for that? It looks like the playerStatus.mode isn't > necessary, since it will never (?) call this routine unless the > ScreenSaver timeout has occurred, right? > I'll have to check your code updates to know for sure, but it sounds like it shouldn't check playserStatus.mode in this case. I've been busy with some comboxbox implementations for color attributes in web ui and the spectrummeter item during the last days, so I haven't had the chance to look into SDT in detail yet. Hopefully I should get some time later in the week. One more question, how are the sdt weather images handled in your implementation. Does it only get the image for the current day or is it possible to configure if it should get the image for today, tomorrow or the day after that. What about the sdt weather text items, do they only look at current day or can you show weather for the next days too ? Regarding sport/game scores I'm starting to think they would be best to handle as a separate item type so I think we can ignore them for now, it's better to get something up and running for the weather info. -- erland Erland Isaksson 'My homepage' (http://erland.isaksson.info) (Install my plugins through Extension Downloader) (Developer of 'TrackStat, SQLPlayList, DynamicPlayList, Custom Browse, Custom Scan, Custom Skip, Multi Library, Title Switcher and Database Query plugins' (http://wiki.erland.isaksson.info/index.php/Category:SlimServer)) ------------------------------------------------------------------------ erland's Profile: http://forums.slimdevices.com/member.php?userid=3124 View this thread: http://forums.slimdevices.com/showthread.php?t=77864 _______________________________________________ jive mailing list [email protected] http://lists.slimdevices.com/mailman/listinfo/jive
