service.upnext icon indicating copy to clipboard operation
service.upnext copied to clipboard

Published 20 hours ago • verified publisher icon im85288

No notification pop-up appears with Venom

Open ShaVerZ opened this issue 5 years ago • 71 comments

FireTV Box (gen.2) 4k Kodi 18.5 Up Next 1.0.7 (Latest)

No notification popup and it seems to have stopped going on to the next episode. The playlist seems to be created within the addon, when viewing the playlist manager.

I couldnt find the exact information you required but the .log link is;

https://pastebin.com/GPvBYptu

I hope this helps and that I've provided what you need.

ShaVerZ avatar Dec 10 '19 12:12 ShaVerZ

Ok, so it appears we did not get any information from the add-on. I would have expected something like this in the logs:

2019-12-10 13:42:44.090 T:1655604096  NOTICE: [service.upnext] Api -> addon_data_received called with data {u'play_info': {u'video_id': u'vid-e16b0168-5ab7-455d-a00f-18985bb3d4a7'}, u'next_episode': {u'rating': None, u'tvshowid': u'622902452527', u'episode': 2, u'title': u'Haar klasgenoten nemen Caro niet serieus', u'season': 1, u'plot': u'[COLOR blue][B]8 more months[/B] available[/COLOR]\n\nEen aantal weken zijn verstreken en Caro heeft langzaam maar zeker de weg op haar nieuwe school gevonden. Yemi en Kyona zijn twee goeie vrienden geworden maar er is iets dat ze desondanks nog steeds voor iedereen achterhoudt.\n\xa0\nAan de hand van haar po\xebzieopdracht voor de les Nederlands probeert ze haar klas op de hoogte te brengen van de situatie thuis. Helaas wordt haar gedicht niet door iedereen serieus genomen en zet Caro het op een lopen.\n\n[COLOR yellow]vrtnu.be/p.6wAOQBler[/COLOR]', u'episodeid': u'622899164527', u'firstaired': u'2019-08-01', u'art': {u'tvshow.fanart': u'https://images.vrt.be/orig/2019/01/07/cd9c3144-12a5-11e9-abcc-02b7b76bf47f.jpg', u'thumb': u'https://images.vrt.be/orig/2019/01/15/2ea66322-18a4-11e9-abcc-02b7b76bf47f.jpg', u'tvshow.clearart': None, u'tvshow.landscape': u'https://images.vrt.be/orig/2019/01/15/2ea66322-18a4-11e9-abcc-02b7b76bf47f.jpg', u'tvshow.poster': u'https://images.vrt.be/orig/2019/01/15/2ea66322-18a4-11e9-abcc-02b7b76bf47f.jpg', u'tvshow.clearlogo': None}, u'playcount': None, u'runtime': 1560, u'showtitle': u'#LikeMe'}, 'id': u'plugin.video.vrt.nu_play_action', u'current_episode': {u'rating': 0.0, u'tvshowid': u'622902452527', u'episode': 1, u'title': u'Caroline moet naar een nieuwe school', u'season': 1, u'plot': u"[COLOR blue][B]8 more months[/B] available[/COLOR]\n\n1 september, de aanvang van een nieuw schooljaar. En dat is het zeker voor nieuweling Caroline Timmers die haar eerste dag start op een, voor haar, compleet onbekende school. \n\xa0\nDoor de ziekte van Caro's moeder Kristel moest haar gezin noodgedwongen verhuizen van hun woonplaats in de Ardennen naar Antwerpen. Een serieuze verandering voor de jonge meid die letterlijk alles heeft moeten achterlaten. \n\xa0\nDat het de overgang niet van een leien dakje zal lopen, wordt al gauw duidelijk.\n\n[COLOR yellow]vrtnu.be/p.9q1okYWGJ[/COLOR]", u'episodeid': u'622899157527', u'firstaired': u'2019-08-01', u'art': {u'tvshow.fanart': u'https://images.vrt.be/orig/2019/01/07/cd9c3144-12a5-11e9-abcc-02b7b76bf47f.jpg', u'thumb': u'https://images.vrt.be/orig/2019/01/07/fbc7b11b-12a6-11e9-abcc-02b7b76bf47f.jpg', u'tvshow.clearart': None, u'tvshow.landscape': u'https://images.vrt.be/orig/2019/01/07/fbc7b11b-12a6-11e9-abcc-02b7b76bf47f.jpg', u'tvshow.poster': u'https://images.vrt.be/orig/2019/01/07/fbc7b11b-12a6-11e9-abcc-02b7b76bf47f.jpg', u'tvshow.clearlogo': None}, u'playcount': 9, u'runtime': 1560.0001220703125, u'showtitle': u'#LikeMe'}}

If UpNext doesn't get any information from the add-on, it will not show a pop-up.

This issue has been reported now a few times, and it was reported it worked with v1.0.1, but not with v1.0.2 or newer. Given that the only changes to v1.0.2 are related to:

  • JSONRPC abstraction (low impact)
  • Playlist handling (high impact)
  • Add endtime property (low impact)

So I would need to know what the add-on is sending, and why we aren't receiving it. cc @Venom123

PS Mind you that a few times now it was suggested this was related to UpNext no longer using Addon Signals, but that is incorrect. Even v1.0.1 is not using Addon Signals, it was removed earlier (39d3661af00e28bfc7613b6d62264ae0caa149cf) in v.0.0.9. The issue related to Addon Signals switching from hex to base64 encoding has been dealt with in the v1.0.7 release. (older versions of UpNext should work fine with older versions of Addon Signals though)

dagwieers avatar Dec 10 '19 12:12 dagwieers

I've tried other versions and just gone back to 1.0.1 and still no popup and not moving to next episode.

v1.0.1 log;

https://pastebin.com/AZ1uxBm7

ShaVerZ avatar Dec 10 '19 13:12 ShaVerZ

So what ought to work

  • Up Next v1.0.6 and older with Addon Signals v0.0.3 and older
  • Up Next v1.0.7 (any Addon Signals version)

But what will definitely not work is Up Next v1.0.6 or older together with Addon Signals v0.0.4 (latest).

dagwieers avatar Dec 10 '19 14:12 dagwieers

Not quite sure what you mean but 1.0.7 doesn't work with any version of Addon Signals (latest).

I'm probably not understanding what you mean.

ShaVerZ avatar Dec 10 '19 14:12 ShaVerZ

Addon Signals v0.0.3 uses hex encoding. Addon Signals v0.0.4 uses base64 encoding.

Up Next v1.06 and older uses hex encoding. Up Next v1.0.7 and newer supports both base64 and hex encoding.

So if you test anything but v1.0.7, you have to use Addon Signals v.0.0.3 otherwise it is guaranteed to not work and we haven't established anything but that.

In other words, the tests you did before using older versions of Up Next don't prove anything unless you also used an older version of Addon Signals.

dagwieers avatar Dec 10 '19 14:12 dagwieers

I was using 1.0.7 when I reported this.....

Thanks for your help though, much appreciated. Probably best I leave it.....

ShaVerZ avatar Dec 10 '19 14:12 ShaVerZ

Not quite sure what you mean but 1.0.7 doesn't work with any version of Addon Signals (latest).

Up Next v1.0.7 works with e.g. the Netflix add-on which is using Addon Signals v0.0.4, so it does work fine (and I am using it daily). We also have integration tests testing Addon Signals v0.0.4 with Up Next to ensure they are compatible. So no issues there. The reported problem here seems to be specific to the implementation of Venom and Seren.

dagwieers avatar Dec 10 '19 16:12 dagwieers

Ah right, I understand. I thought you meant I should try 1.0.7 ensuring Addon Signals 0.0.4, that's where the confusion lay. I thought you meant it should work with that combination whilst using Seren/Venom.

It does seem to work with old versions of Kodi though.....

Thanks again.

ShaVerZ avatar Dec 10 '19 16:12 ShaVerZ

@ShaVerZ Well, it should work for Venom and Seren, but it clearly doesn't. We haven't established the reason for this. I was told it stopped working even with v1.0.2 up to v1.0.6, which would indicate an issue with any of these changes: https://github.com/im85288/service.upnext/compare/1.0.1..1.0.2

But I am not sure if the tests were executed with the correct Addon Signals v0.0.3, so not clear if we can make that conclusion.

dagwieers avatar Dec 10 '19 16:12 dagwieers

If theres anything you want me to try, let me know, I have a few spare FireTV boxes which are not in use. I'm happy to test certain setups if you wish.....

ShaVerZ avatar Dec 10 '19 17:12 ShaVerZ

Well, it's working now. I had to drop back to Kodi 18.2 though, no idea why it seems fine with 18.2 but nothing higher..... thought it was worth mentioning.

ShaVerZ avatar Dec 11 '19 13:12 ShaVerZ

So using Kodi 18.2 and what version of UpNext and AddonSignals?

dagwieers avatar Dec 11 '19 13:12 dagwieers

I'm not currently home but I could only get 0.1.8 working. v1.0.1 and up didnt. Im assuming it is addon Signals v0.0.3 but I'll need to check when I'm home.

ShaVerZ avatar Dec 11 '19 14:12 ShaVerZ

Looking at the Venom code I see various reasons why it would fail: https://github.com/123Venom/plugin.video.venom/blob/84edfdb37ecd785b5f0dac472f859c45d0c94fa2/resources/lib/modules/player.py#L347-L370

Apart from the fact that control.playlist.getposition() and self.mediatype may interfere with normal operation, the broad try-blocks may cause things not completing as expected.

Proper python code should not be using broad excepts, and I would suggest that Venom uses tox and pylint for testing for common python issues. Something that worked before could suddenly fail because of changes in behaviour that was not accounted for. Especially Python is susceptible for this. cc @123Venom

That said, I don't know if this is the reason for Up Next not working. Adding more log-statements that would give information on what is going on would be helpful as well. Now I don't see any logging from Venom in the logs. Did it ever send out the notification? I wouldn't know.

dagwieers avatar Dec 11 '19 16:12 dagwieers

I got back earlier and noticed it wasnt working, I've tried a combination of installs and none seem to work, I can only assume that Venom has received an update at some point and it has stopped up next working. It doesnt matter which combination you use re: Add on Signals.

I could go back and try older versions of Venom but tbh, I cant be assed today ;)

ShaVerZ avatar Dec 11 '19 16:12 ShaVerZ

IMO it is better to wait for the developer @123Venom to troubleshoot Venom.

dagwieers avatar Dec 11 '19 17:12 dagwieers

ok I've been looking at this and I'm stumped how I can run v1.0.1 of UpNext without a single issue but both Seren and Venom suffer the same exact problem beyond v1.0.1 with no UpNext notification showing up. I just tried v1.0.7 and logged this 2019-12-12 03:22:30.045 T:7332 NOTICE: [service.upnext] Api -> addon_data_received called with data {u'play_info': {u'item_id': 2244839}, u'next_episode': {u'rating': u'7.5', u'tvshowid': u'80379', u'art': {u'tvshow.fanart': u'https://thetvdb.com/banners/fanart/original/80379-34.jpg', u'thumb': u'https://thetvdb.com/banners/episodes/80379/5714255.jpg', u'tvshow.clearart': u'https://assets.fanart.tv/fanart/tv/80379/hdclearart/the-big-bang-theory-519818ab2d8a9.png', u'tvshow.landscape': u'https://thetvdb.com/banners/fanart/original/80379-34.jpg', u'tvshow.poster': u'https://thetvdb.com/banners/posters/80379-18.jpg', u'tvshow.clearlogo': u'https://assets.fanart.tv/fanart/tv/80379/hdtvlogo/the-big-bang-theory-50477a642f672.png'}, u'plot': u'The Air Force shows interest in the guidance system and Penny lets it slip at work that Bernadette is expecting.', u'season': u'10', u'title': u'The Military Miniaturization', u'episodeid': 2267810, u'tvshowimdb': u'tt0898266', u'firstaired': u'2007', u'year': u'2016', u'episode': u'2', u'playcount': 0, u'showtitle': u'The Big Bang Theory'}, u'notification_time': 60, 'id': u'plugin.video.venom_play_action', u'current_episode': {u'rating': u'7.6', u'tvshowid': u'80379', u'art': {u'tvshow.fanart': u'https://thetvdb.com/banners/fanart/original/80379-34.jpg', u'thumb': u'https://thetvdb.com/banners/episodes/80379/5654968.jpg', u'tvshow.clearart': u'https://assets.fanart.tv/fanart/tv/80379/hdclearart/the-big-bang-theory-519818ab2d8a9.png', u'tvshow.landscape': u'https://thetvdb.com/banners/fanart/original/80379-34.jpg', u'tvshow.poster': u'https://thetvdb.com/banners/posters/80379-18.jpg', u'tvshow.clearlogo': u'https://assets.fanart.tv/fanart/tv/80379/hdtvlogo/the-big-bang-theory-50477a642f672.png'}, u'plot': u"After Leonard's father, Alfred and Sheldon's mother, Mary share an evening together, everyone deals with an awkward morning the following day. Also, Penny's family arrives for the ceremony, including her anxiety-ridden mother, Susan, and her drug dealing brother, Randall.", u'season': u'10', u'title': u'The Conjugal Conjecture', u'episodeid': 2244839, u'firstaired': u'2007', u'episode': u'1', u'playcount': 0, u'showtitle': u'The Big Bang Theory'}} 2019-12-12 03:22:35.055 T:10360 WARNING: CRenderManager::WaitForBuffer - timeout waiting for buffer 2019-12-12 03:23:35.179 T:7332 WARNING: Previous line repeats 1 times. 2019-12-12 03:23:35.179 T:7332 NOTICE: [service.upnext] Monitor -> Calling autoplayback totaltime - playtime is 48.3489990234 2019-12-12 03:23:35.195 T:7332 NOTICE: [service.upnext] Api -> Got details of next playlist item {u'jsonrpc': u'2.0', u'id': 1, u'result': {u'items': [{u'plot': u'The Air Force shows interest in the guidance system and Penny lets it slip at work that Bernadette is expecting.', u'tvshowid': -1, u'dateadded': u'', u'episode': 2, u'rating': 7.5, u'season': 10, u'lastplayed': u'', u'resume': {u'position': 0.0, u'total': 0.0}, u'label': u'10x02 . The Military Miniaturization', u'writer': [u'Chuck Lorre', u'Eric Kaplan', u'Jim Reynolds', u'Maria Ferrari', u'Steve Holland', u'Steven Molaro'], u'firstaired': u'', u'title': u'The Military Miniaturization', u'file': u'plugin://plugin.video.venom/?action=play&title=The+Military+Miniaturization&year=2007&imdb=tt0898266&tvdb=80379&season=10&episode=2&tvshowtitle=The+Big+Bang+Theory&premiered=2016-09-26&meta=%7B%22rating%22%3A+%227.5%22%2C+%22poster2%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Ftvposter%2Fthe-big-bang-theory-5214251a5b671.jpg%22%2C+%22imdb%22%3A+%22tt0898266%22%2C+%22year%22%3A+%222016%22%2C+%22duration%22%3A+%221500%22%2C+%22landscape%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Ftvthumb%2FT_80379+%280%29.jpg%22%2C+%22plot%22%3A+%22The+Air+Force+shows+interest+in+the+guidance+system+and+Penny+lets+it+slip+at+work+that+Bernadette+is+expecting.%22%2C+%22votes%22%3A+%2241886%22%2C+%22castandart%22%3A+%5B%7B%22role%22%3A+%22Sheldon+Cooper%22%2C+%22name%22%3A+%22Jim+Parsons%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325467.jpg%22%7D%2C+%7B%22role%22%3A+%22Leonard+Hofstadter%22%2C+%22name%22%3A+%22Johnny+Galecki%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325466.jpg%22%7D%2C+%7B%22role%22%3A+%22Penny%22%2C+%22name%22%3A+%22Kaley+Cuoco%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325465.jpg%22%7D%2C+%7B%22role%22%3A+%22Rajesh+Koothrappali%22%2C+%22name%22%3A+%22Kunal+Nayyar%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325463.jpg%22%7D%2C+%7B%22role%22%3A+%22Howard+Wolowitz%22%2C+%22name%22%3A+%22Simon+Helberg%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325464.jpg%22%7D%2C+%7B%22role%22%3A+%22Bernadette+Rostenkowski%22%2C+%22name%22%3A+%22Melissa+Rauch%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325462.jpg%22%7D%2C+%7B%22role%22%3A+%22Amy+Farrah+Fowler%22%2C+%22name%22%3A+%22Mayim+Bialik%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325461.jpg%22%7D%2C+%7B%22role%22%3A+%22Emily+Sweeney%22%2C+%22name%22%3A+%22Laura+Spencer%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F358188.jpg%22%7D%2C+%7B%22role%22%3A+%22Leslie+Winkle%22%2C+%22name%22%3A+%22Sara+Gilbert%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F378343.jpg%22%7D%2C+%7B%22role%22%3A+%22Stuart+Bloom%22%2C+%22name%22%3A+%22Kevin+Sussman%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F378344.jpg%22%7D%2C+%7B%22role%22%3A+%22Barry+Kripke%22%2C+%22name%22%3A+%22John+Ross+Bowie%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F382566.jpg%22%7D%2C+%7B%22role%22%3A+%22Debbie+Wolowitz%22%2C+%22name%22%3A+%22Carol+Ann+Susi%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F403186.jpg%22%7D%5D%2C+%22title%22%3A+%22The+Military+Miniaturization%22%2C+%22tvdb%22%3A+%2280379%22%2C+%22mpaa%22%3A+%22TV-14%22%2C+%22writer%22%3A+%22Chuck+Lorre+%2F+Eric+Kaplan+%2F+Jim+Reynolds+%2F+Maria+Ferrari+%2F+Steve+Holland+%2F+Steven+Molaro%22%2C+%22label%22%3A+%22The+Military+Miniaturization%22%2C+%22episodeIDS%22%3A+%7B%22tvdb%22%3A+5714255%2C+%22tmdb%22%3A+1221932%2C+%22imdb%22%3A+%22tt5973910%22%2C+%22tvrage%22%3A+0%2C+%22trakt%22%3A+2267810%7D%2C+%22season%22%3A+%2210%22%2C+%22banner2%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Ftvbanner%2Fthe-big-bang-theory-52db1f74e2bee.jpg%22%2C+%22tvshowyear%22%3A+%222007%22%2C+%22fanart2%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Fshowbackground%2Fthe-big-bang-theory-561e3c3dbc1ea.jpg%22%2C+%22status%22%3A+%22Ended%22%2C+%22extended%22%3A+true%2C+%22poster%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Fposters%2F80379-18.jpg%22%2C+%22tvshowtitle%22%3A+%22The+Big+Bang+Theory%22%2C+%22mediatype%22%3A+%22episode%22%2C+%22studio%22%3A+%22CBS%22%2C+%22genre%22%3A+%22Comedy%22%2C+%22banner%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Fgraphical%2F80379-g13.jpg%22%2C+%22clearart%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Fhdclearart%2Fthe-big-bang-theory-519818ab2d8a9.png%22%2C+%22thumb%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Fepisodes%2F80379%2F5714255.jpg%22%2C+%22episode%22%3A+%222%22%2C+%22metacache%22%3A+true%2C+%22overlay%22%3A+6%2C+%22premiered%22%3A+%222016-09-26%22%2C+%22clearlogo%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Fhdtvlogo%2Fthe-big-bang-theory-50477a642f672.png%22%2C+%22fanart%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Ffanart%2Foriginal%2F80379-34.jpg%22%2C+%22season_poster%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Fposters%2F80379-18.jpg%22%2C+%22playcount%22%3A+0%2C+%22seasoncount%22%3A+24%2C+%22trailer%22%3A+%22plugin%3A%2F%2Fplugin.video.venom%2F%3Faction%3Dtrailer%26name%3DThe%2BBig%2BBang%2BTheory%26imdb%3Dtt0898266%22%7D&t=20191212032118292000', u'art': {u'season.poster': u'image://https%3a%2f%2fthetvdb.com%2fbanners%2fposters%2f80379-18.jpg/', u'fanart': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2fshowbackground%2fthe-big-bang-theory-561e3c3dbc1ea.jpg/', u'poster': u'image://https%3a%2f%2fthetvdb.com%2fbanners%2fposters%2f80379-18.jpg/', u'clearlogo': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2fhdtvlogo%2fthe-big-bang-theory-50477a642f672.png/', u'clearart': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2fhdclearart%2fthe-big-bang-theory-519818ab2d8a9.png/', u'tvshow.poster': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2ftvposter%2fthe-big-bang-theory-5214251a5b671.jpg/', u'banner': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2ftvbanner%2fthe-big-bang-theory-52db1f74e2bee.jpg/', u'landscape': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2ftvthumb%2fT_80379%20(0).jpg/', u'thumb': u'image://https%3a%2f%2fthetvdb.com%2fbanners%2fepisodes%2f80379%2f5714255.jpg/'}, u'playcount': 0, u'runtime': 1500, u'type': u'episode', u'showtitle': u'The Big Bang Theory', u'streamdetails': {u'video': [{u'language': u'', u'stereomode': u'', u'width': 0, u'codec': u'h264', u'aspect': 0.0, u'duration': 0, u'height': 0}], u'audio': [], u'subtitle': []}}], u'limits': {u'start': 1, u'total': 24, u'end': 2}}} 2019-12-12 03:23:35.195 T:7332 NOTICE: [service.upnext] PlaybackManager -> episode details {u'plot': u'The Air Force shows interest in the guidance system and Penny lets it slip at work that Bernadette is expecting.', u'tvshowid': -1, u'dateadded': u'', u'episode': 2, u'rating': 7.5, u'season': 10, u'lastplayed': u'', u'episodeid': None, u'resume': {u'position': 0.0, u'total': 0.0}, u'label': u'10x02 . The Military Miniaturization', u'writer': [u'Chuck Lorre', u'Eric Kaplan', u'Jim Reynolds', u'Maria Ferrari', u'Steve Holland', u'Steven Molaro'], u'firstaired': u'', u'title': u'The Military Miniaturization', u'file': u'plugin://plugin.video.venom/?action=play&title=The+Military+Miniaturization&year=2007&imdb=tt0898266&tvdb=80379&season=10&episode=2&tvshowtitle=The+Big+Bang+Theory&premiered=2016-09-26&meta=%7B%22rating%22%3A+%227.5%22%2C+%22poster2%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Ftvposter%2Fthe-big-bang-theory-5214251a5b671.jpg%22%2C+%22imdb%22%3A+%22tt0898266%22%2C+%22year%22%3A+%222016%22%2C+%22duration%22%3A+%221500%22%2C+%22landscape%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Ftvthumb%2FT_80379+%280%29.jpg%22%2C+%22plot%22%3A+%22The+Air+Force+shows+interest+in+the+guidance+system+and+Penny+lets+it+slip+at+work+that+Bernadette+is+expecting.%22%2C+%22votes%22%3A+%2241886%22%2C+%22castandart%22%3A+%5B%7B%22role%22%3A+%22Sheldon+Cooper%22%2C+%22name%22%3A+%22Jim+Parsons%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325467.jpg%22%7D%2C+%7B%22role%22%3A+%22Leonard+Hofstadter%22%2C+%22name%22%3A+%22Johnny+Galecki%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325466.jpg%22%7D%2C+%7B%22role%22%3A+%22Penny%22%2C+%22name%22%3A+%22Kaley+Cuoco%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325465.jpg%22%7D%2C+%7B%22role%22%3A+%22Rajesh+Koothrappali%22%2C+%22name%22%3A+%22Kunal+Nayyar%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325463.jpg%22%7D%2C+%7B%22role%22%3A+%22Howard+Wolowitz%22%2C+%22name%22%3A+%22Simon+Helberg%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325464.jpg%22%7D%2C+%7B%22role%22%3A+%22Bernadette+Rostenkowski%22%2C+%22name%22%3A+%22Melissa+Rauch%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325462.jpg%22%7D%2C+%7B%22role%22%3A+%22Amy+Farrah+Fowler%22%2C+%22name%22%3A+%22Mayim+Bialik%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325461.jpg%22%7D%2C+%7B%22role%22%3A+%22Emily+Sweeney%22%2C+%22name%22%3A+%22Laura+Spencer%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F358188.jpg%22%7D%2C+%7B%22role%22%3A+%22Leslie+Winkle%22%2C+%22name%22%3A+%22Sara+Gilbert%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F378343.jpg%22%7D%2C+%7B%22role%22%3A+%22Stuart+Bloom%22%2C+%22name%22%3A+%22Kevin+Sussman%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F378344.jpg%22%7D%2C+%7B%22role%22%3A+%22Barry+Kripke%22%2C+%22name%22%3A+%22John+Ross+Bowie%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F382566.jpg%22%7D%2C+%7B%22role%22%3A+%22Debbie+Wolowitz%22%2C+%22name%22%3A+%22Carol+Ann+Susi%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F403186.jpg%22%7D%5D%2C+%22title%22%3A+%22The+Military+Miniaturization%22%2C+%22tvdb%22%3A+%2280379%22%2C+%22mpaa%22%3A+%22TV-14%22%2C+%22writer%22%3A+%22Chuck+Lorre+%2F+Eric+Kaplan+%2F+Jim+Reynolds+%2F+Maria+Ferrari+%2F+Steve+Holland+%2F+Steven+Molaro%22%2C+%22label%22%3A+%22The+Military+Miniaturization%22%2C+%22episodeIDS%22%3A+%7B%22tvdb%22%3A+5714255%2C+%22tmdb%22%3A+1221932%2C+%22imdb%22%3A+%22tt5973910%22%2C+%22tvrage%22%3A+0%2C+%22trakt%22%3A+2267810%7D%2C+%22season%22%3A+%2210%22%2C+%22banner2%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Ftvbanner%2Fthe-big-bang-theory-52db1f74e2bee.jpg%22%2C+%22tvshowyear%22%3A+%222007%22%2C+%22fanart2%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Fshowbackground%2Fthe-big-bang-theory-561e3c3dbc1ea.jpg%22%2C+%22status%22%3A+%22Ended%22%2C+%22extended%22%3A+true%2C+%22poster%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Fposters%2F80379-18.jpg%22%2C+%22tvshowtitle%22%3A+%22The+Big+Bang+Theory%22%2C+%22mediatype%22%3A+%22episode%22%2C+%22studio%22%3A+%22CBS%22%2C+%22genre%22%3A+%22Comedy%22%2C+%22banner%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Fgraphical%2F80379-g13.jpg%22%2C+%22clearart%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Fhdclearart%2Fthe-big-bang-theory-519818ab2d8a9.png%22%2C+%22thumb%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Fepisodes%2F80379%2F5714255.jpg%22%2C+%22episode%22%3A+%222%22%2C+%22metacache%22%3A+true%2C+%22overlay%22%3A+6%2C+%22premiered%22%3A+%222016-09-26%22%2C+%22clearlogo%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Fhdtvlogo%2Fthe-big-bang-theory-50477a642f672.png%22%2C+%22fanart%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Ffanart%2Foriginal%2F80379-34.jpg%22%2C+%22season_poster%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Fposters%2F80379-18.jpg%22%2C+%22playcount%22%3A+0%2C+%22seasoncount%22%3A+24%2C+%22trailer%22%3A+%22plugin%3A%2F%2Fplugin.video.venom%2F%3Faction%3Dtrailer%26name%3DThe%2BBig%2BBang%2BTheory%26imdb%3Dtt0898266%22%7D&t=20191212032118292000', u'art': {u'season.poster': u'image://https%3a%2f%2fthetvdb.com%2fbanners%2fposters%2f80379-18.jpg/', u'fanart': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2fshowbackground%2fthe-big-bang-theory-561e3c3dbc1ea.jpg/', u'poster': u'image://https%3a%2f%2fthetvdb.com%2fbanners%2fposters%2f80379-18.jpg/', u'clearlogo': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2fhdtvlogo%2fthe-big-bang-theory-50477a642f672.png/', u'clearart': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2fhdclearart%2fthe-big-bang-theory-519818ab2d8a9.png/', u'tvshow.poster': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2ftvposter%2fthe-big-bang-theory-5214251a5b671.jpg/', u'banner': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2ftvbanner%2fthe-big-bang-theory-52db1f74e2bee.jpg/', u'landscape': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2ftvthumb%2fT_80379%20(0).jpg/', u'thumb': u'image://https%3a%2f%2fthetvdb.com%2fbanners%2fepisodes%2f80379%2f5714255.jpg/'}, u'playcount': 0, u'runtime': 1500, u'type': u'episode', u'showtitle': u'The Big Bang Theory', u'streamdetails': {u'video': [{u'language': u'', u'stereomode': u'', u'width': 0, u'codec': u'h264', u'aspect': 0.0, u'duration': 0, u'height': 0}], u'audio': [], u'subtitle': []}} 2019-12-12 03:23:35.195 T:7332 NOTICE: [service.upnext] Monitor -> Up Next style autoplay succeeded. but not notification. I'm seeing jasonrpc so I have to wonder if UpNext receives the proper data why it's looking in the local library? It never did that before unless tvshowid was not received but it is. I've made zero changes to the data sent and received to UpNext since implementing support with v1.0.1 yet anything beyond 1.0.1 and it no longer works in Seren or Venom. I'm going to revert back to working v1.0.1 and compare logged data

123Venom avatar Dec 12 '19 09:12 123Venom

Which version of Kodi have you tested @123Venom and which version of Addon Signals?

I cant get any version to work on FireTV Box (gen2).....

ShaVerZ avatar Dec 12 '19 09:12 ShaVerZ

v1.0.1 log...works perfectly! I see what's not transmitted but why idk 2019-12-12 04:09:35.438 T:9520 NOTICE: UP NEXT Monitor -> log : Calling autoplayback totaltime - playtime is 59.3669433594 2019-12-12 04:09:35.441 T:9520 NOTICE: UP NEXT Api -> log : handle_addon_lookup_of_next_episode episode returning data {"rating": "7.7", "tvshowid": "80379", "art": {"tvshow.fanart": "https://thetvdb.com/banners/fanart/original/80379-34.jpg", "thumb": "https://thetvdb.com/banners/episodes/80379/4413988.jpg", "tvshow.clearart": "https://assets.fanart.tv/fanart/tv/80379/hdclearart/the-big-bang-theory-519818ab2d8a9.png", "tvshow.landscape": "https://thetvdb.com/banners/fanart/original/80379-34.jpg", "tvshow.poster": "https://thetvdb.com/banners/posters/80379-18.jpg", "tvshow.clearlogo": "https://assets.fanart.tv/fanart/tv/80379/hdtvlogo/the-big-bang-theory-50477a642f672.png"}, "plot": "A game threatens Sheldon's relationship with Stephen Hawking; Penny secretly signs up for a college class.", "season": "6", "title": "The Extract Obliteration", "episodeid": 75485, "tvshowimdb": "tt0898266", "firstaired": "2007", "year": "2012", "episode": "6", "playcount": 0, "showtitle": "The Big Bang Theory"} 2019-12-12 04:09:35.443 T:9520 NOTICE: UP NEXT Api -> log : handle_addon_lookup_of_current episode returning data {"rating": "7.7", "tvshowid": "80379", "art": {"tvshow.fanart": "https://thetvdb.com/banners/fanart/original/80379-34.jpg", "thumb": "https://thetvdb.com/banners/episodes/80379/4408722.jpg", "tvshow.clearart": "https://assets.fanart.tv/fanart/tv/80379/hdclearart/the-big-bang-theory-519818ab2d8a9.png", "tvshow.landscape": "https://thetvdb.com/banners/fanart/original/80379-34.jpg", "tvshow.poster": "https://thetvdb.com/banners/posters/80379-18.jpg", "tvshow.clearlogo": "https://assets.fanart.tv/fanart/tv/80379/hdtvlogo/the-big-bang-theory-50477a642f672.png"}, "plot": "The gang heads to Stuart's comic book store to celebrate Halloween; Leonard uses science to seduce Penny.", "season": "6", "title": "The Holographic Excitation", "episodeid": 75484, "firstaired": "2007", "episode": "5", "playcount": 0, "showtitle": "The Big Bang Theory"} 2019-12-12 04:09:35.446 T:9520 NOTICE: UP NEXT PlaybackManager -> log : episode details {"rating": "7.7", "tvshowid": "80379", "art": {"tvshow.fanart": "https://thetvdb.com/banners/fanart/original/80379-34.jpg", "thumb": "https://thetvdb.com/banners/episodes/80379/4413988.jpg", "tvshow.clearart": "https://assets.fanart.tv/fanart/tv/80379/hdclearart/the-big-bang-theory-519818ab2d8a9.png", "tvshow.landscape": "https://thetvdb.com/banners/fanart/original/80379-34.jpg", "tvshow.poster": "https://thetvdb.com/banners/posters/80379-18.jpg", "tvshow.clearlogo": "https://assets.fanart.tv/fanart/tv/80379/hdtvlogo/the-big-bang-theory-50477a642f672.png"}, "plot": "A game threatens Sheldon's relationship with Stephen Hawking; Penny secretly signs up for a college class.", "season": "6", "title": "The Extract Obliteration", "episodeid": 75485, "tvshowimdb": "tt0898266", "firstaired": "2007", "year": "2012", "episode": "6", "playcount": 0, "showtitle": "The Big Bang Theory"} 2019-12-12 04:09:35.454 T:9520 NOTICE: UP NEXT PlaybackManager -> log : played in a row settings "3" 2019-12-12 04:09:35.457 T:9520 NOTICE: UP NEXT PlaybackManager -> log : played in a row 1 2019-12-12 04:09:35.459 T:9520 NOTICE: UP NEXT PlaybackManager -> log : showing next up page as played in a row is 1 2019-12-12 04:10:26.147 T:14156 WARNING: CGUIWindowManager - CGUIWindowManager::HandleAction - ignoring action 107, because topmost modal dialog closing animation is running 2019-12-12 04:10:26.166 T:9520 WARNING: Previous line repeats 1 times. 2019-12-12 04:10:26.166 T:9520 NOTICE: UP NEXT PlaybackManager -> log : playing media episode 2019-12-12 04:10:26.168 T:9520 NOTICE: UP NEXT Api -> log : sending data to addon to play: {"item_id": 75484} 2019-12-12 04:10:26.181 T:14156 WARNING: CGUIWindowManager - CGUIWindowManager::HandleAction - ignoring action 107, because topmost modal dialog closing animation is running 2019-12-12 04:10:26.183 T:9520 NOTICE: UP NEXT Monitor -> log : Up Next style autoplay succeeded.

https://postimg.cc/rDnpsQ5W

123Venom avatar Dec 12 '19 09:12 123Venom

Which version of Kodi have you tested @123Venom and which version of Addon Signals?

I cant get any version to work on FireTV Box (gen2).....

When I implemented UpNext support I was on 18.1...currently on 18.5 and just tested on win pc and Shield and both behave the same way. Anything after v1.0.1 and no notification. 1.0.1 works flawlessly and I've changed no code since nor had reason to till perhaps now. I'd love to get to the bottom of this. And Addon Signals I'm still on 0.0.3 with auto updates off because when that updated to 0.0.4 it also broke but going back to 0.0.3 and all was fine.

123Venom avatar Dec 12 '19 09:12 123Venom

@123Venom So you confirm with 100% certainty that v1.0.1 works fine, and v1.0.2 does not?

Can you please post full Kodi debug logs rather than fragments?

Also previous reports did not include the addon_data_received called with data section, which indicated there was no information send. So it is absolutely essential that we can interpret the information we get correctly, and if you only provide us with fragments, we may be looking at the wrong data and draw wrong conclusions.

dagwieers avatar Dec 12 '19 12:12 dagwieers

SEREN has had an update and Up Next Works, I still can't get it to work with Venom.

However, I'm using v1.0.1 with 0.0.0.3 Addon Signals.

ShaVerZ avatar Dec 13 '19 12:12 ShaVerZ

@ShaVerZ Sigh, so you are now confirming that Venom does not work with Up Next v1.0.1 using Venom with Addon Signals v0.0.3 ? This would go against all previous reports...

dagwieers avatar Dec 13 '19 12:12 dagwieers

When forwarding to the end of a programme, to see if Up Next works, should you allow a minute or two to the end or longer.

I used Seren last night and watched a programme completely and it worked. Perhaps I'm not leaving enough time at the end.....

ShaVerZ avatar Dec 13 '19 12:12 ShaVerZ

According to the Seren changelog, Seren v1.6.0 removed Up Next support

  • Removed support for UpNext addon

dagwieers avatar Dec 13 '19 12:12 dagwieers

@ShaVerZ Up Next by default is configured to show the pop-up 30 seconds before the end. The master branch now has some heuristics that make this time depend on the length of the video. from 15 seconds before the end to 50 seconds (but configurable to anything you prefer).

Update: Some add-ons provide Up Next with their own notification time, I noticed Seren has its own configuration option which also defaults to 30 seconds (but is a slider from 10 to 60).

dagwieers avatar Dec 13 '19 12:12 dagwieers

Well, its 100% working with Seren and I have some settings which resemble Up Next settings.

ShaVerZ avatar Dec 13 '19 12:12 ShaVerZ

20191213_123159 20191213_123218

ShaVerZ avatar Dec 13 '19 12:12 ShaVerZ

I just tested Venom and I can't get it working with v1.0.1 and 0.0.3.

Perhaps Seren is using it's own notifications instead of Up Next.

ShaVerZ avatar Dec 13 '19 12:12 ShaVerZ

Well, the Seren code at https://github.com/nixgates/plugin.video.seren is incomplete. The code refers to files that do not exist in the tree. But it is clear they moved to their own implementation rather than use Up Next.

dagwieers avatar Dec 13 '19 12:12 dagwieers