[Razze]

so, um, what happens if xbmc has the right show data, but when scrobbled trakt.tv has the wrong show?

this should be pretty much impossible if your using the tvdb scrapper, if you are feel free to head over to github and open an issue on the problem would be nice if you could provide the name of the show and episode too

Forgot to remove that setting. Retrying won't help you if the server is not responding at all.

No harm

[...]

Just wanted to reply and say the new script seems to be working quite well. The Debugging to the log file seems a bit sparse for my taste. Is there a higher debug level, something like a trace level of DEBUG? I am a python developer by trade (sadly stuck in 2.7 due to legacy requirements). I'm very familiar with the libraries for requests, urlib, urllib2, OAuth, etc, but not the xbmc/kodi api library. If I had some higher debugging, I might be able to help out some (or had some time to learn the Kodi library). Is there a bug tracker or any feature todo list I could scan over and possibly contribute?

Retrying does hold a certain harm, especially on post/delete/put requests, as trakt is not providing transactions.
And spamming their servers won't help you, but there is a inbuild retry on the trakt.py level.

As far as I know it's not possible to use log levels the way their intended to be used. The kodi team suggests every addon to log on the logdebug level and that level only. So no, there is no other log :/

If you want to help out check these issues and githubs:

https://github.com/fuzeman/trakt.py/tree/develop/
https://github.com/fuzeman/trakt.py/issues

https://github.com/Razzeee/script.trakt/tree/dev
https://github.com/Razzeee/script.trakt/issues

Thanks!

Would be greate if some native english speakers could check this commit and add some comments.
Not really sure about the string for "playback progress" etc
https://github.com/Razzeee/script.trakt/...63b1e009e4

Verbage looks good here, but I speak the Picksburghese dialect of English, so I may not be the best to answer

Only question I have when looking at theat commit is "Is there a reason the full exception isn't getting passed to the logger?"

For example, instead of:

except Exception:
logger.debug("[Playback Sync] Invalid trakt.tv progress list, possible error getting data from trakt, aborting trakt.tv playback update.")
return False, False

why not use:

except Exception, e:
logger.debug("[Playback Sync] Invalid trakt.tv progress list, aborting trakt.tv playback update. Error: %s" % str(e))
return False, False

Just curious. It would give more robust error reporting and help troubleshooting errors.

Mostly this are cases where I know it will be a server problem on trakt for 99% of the times this get thrown.
Just haven't seen that bugging me (yet)