Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Plex Server v1.3.x & 3rd Party Agents #98

Closed
Dingmatt opened this issue Dec 8, 2016 · 30 comments
Closed

Plex Server v1.3.x & 3rd Party Agents #98

Dingmatt opened this issue Dec 8, 2016 · 30 comments

Comments

@Dingmatt
Copy link
Collaborator

Dingmatt commented Dec 8, 2016

There seems to be an issue with the latest Plex server version attempting to retrieve saved metadata from non-existent directories (its loading from the wrong place), it only seems to affect new metadata (not existing).

This isn't a Hama bug, just giving you the heads up.

Edit: This issues still present in the public v1.3.2.

@sven-7
Copy link
Contributor

sven-7 commented Dec 8, 2016

Glad to find that out. Was having issues with that this morning. Thanks!

@sven-7
Copy link
Contributor

sven-7 commented Dec 9, 2016

@Dingmatt -- I've had pretty much all of my season posters disappear. Is this happening for you as well?

@Dingmatt
Copy link
Collaborator Author

Dingmatt commented Dec 9, 2016

@sven-7 Yep, that's the issue. You'll also find that your episode thumbnails disappear as well, getting replaced with the low quality screen capture kind.

What seems to be happening is Plex isn't retrieving the images from the correct place, you'll find your error logs filled with warning about it being unable to load files from directories that don't exist.

For the time being I've downgraded to 1.2.7 and done a full refresh which has bought all the images back.

@sven-7
Copy link
Contributor

sven-7 commented Dec 10, 2016

Yeah. Definitely a pain.

Do you know if the episode ordering (my OP issue #66) is part of that too?

@Dingmatt
Copy link
Collaborator Author

Not sure, the ordering is mainly done based on the filenames but that being said if it's then being stored as metadata its possible.

I've seen a few ordering bugs being reported on the Plex forums, something to do with it being based on airdate rather than naming; that might be the cause.

@sven-7
Copy link
Contributor

sven-7 commented Dec 10, 2016

The weird part about OP is the episode titles aren't coming either. The scanner logs look like the scanner freaks out about it too.

@KingJ
Copy link
Collaborator

KingJ commented Dec 10, 2016 via email

@sven-7
Copy link
Contributor

sven-7 commented Dec 10, 2016

Sure -- I posted them here: ZeroQI/Absolute-Series-Scanner#66

@KingJ
Copy link
Collaborator

KingJ commented Dec 11, 2016

For reference, @Dingmatt has also posted this on the Plex Forums here- other 3rd party metadata agents are affected too, e.g. gboudreau/XBMCnfoTVImporter.bundle/issues/84.

I've been able to reproduce this issue on both my production Linux Plex server (v1.3.1) and my development Windows Plex server (v1.3.2).

@Dingmatt
Copy link
Collaborator Author

Unfortunately it looks like 1.3.3 hasn't resolved this issue.

@KingJ
Copy link
Collaborator

KingJ commented Dec 16, 2016

Confirmed, unfortunately. I was hoping they'd have fixed it by now.

@ZeroQI
Copy link
Owner

ZeroQI commented Dec 16, 2016

Updating my synology nas to "PlexMediaServer-1.3.3.3148-b38628e-x86.spk" now but seem like it's not fixed

@ZeroQI
Copy link
Owner

ZeroQI commented Dec 28, 2016

@Dingmatt
Copy link
Collaborator Author

Dingmatt commented Jan 4, 2017

The Plex staff have recently acknowledged this issue on their forums and say they're looking into it.

@Dingmatt
Copy link
Collaborator Author

Dingmatt commented Feb 14, 2017

@KingJ @ZeroQI @sven-7 Just to let you know I've almost completed development on (and testing) a workaround for this issue as it doesn't look like we can rely on the Plex team.

Its not much proof but here's a screenshot of Plex v1.4.1 running v0.1.3 of AMSA with the workaround (yay for metadata); it shouldn't be difficult to sort HAMA as well.
image

@sven-7
Copy link
Contributor

sven-7 commented Feb 14, 2017

Awesome, @Dingmatt! I'd be happy to help test when it's ready.

@Dingmatt
Copy link
Collaborator Author

Dingmatt commented Feb 14, 2017

Ok, I'm testing the workaround on a modified HAMA now...

Edit: Yep it works and screw the Plex dev team.
@KingJ @ZeroQI I have a workaround but your not going to like it.

@Dingmatt
Copy link
Collaborator Author

@KingJ
Copy link
Collaborator

KingJ commented Feb 14, 2017

Ok, that's a frustrating workaround and something that really shouldn't be the case! Thanks for taking the time to track this down.

I got in touch with some of the Plex team directly today to try and get some more attention on this issue. I've had a response so far, albeit not a solution or information on a fix timeline. I'll keep you posted here.

@sven-7
Copy link
Contributor

sven-7 commented Feb 14, 2017

Thanks! Yet, I agree with both of you...frustrating and not ideal.

Hopefully @Dingmatt's solution causes them to take a closer look.

@Dingmatt
Copy link
Collaborator Author

Dingmatt commented Feb 15, 2017

@sven-7 @KingJ @ZeroQI Seems like that workaround was the final piece of the puzzle they needed, they've been updating all the related forum posts saying they've identified the problem code and are working on a fix.

@sven-7
Copy link
Contributor

sven-7 commented Feb 15, 2017

I'm glad they admitted it was an "unfortunate regression".

@KingJ
Copy link
Collaborator

KingJ commented Feb 15, 2017

@Dingmatt Excellent, it's great to hear from them both that they've identified the problematic change and also that breaking 3rd party metadata agents was completely unintentional - I was starting to get a little concerned that silently breaking them was intentional.

Thanks again for tracking down the workaround, seems like it really did help speed up their identification of the cause.

@ghost
Copy link

ghost commented Feb 15, 2017

@Dingmatt do you accept donations somewhere? You're my #1 hero right now. 😭

@ZeroQI
Copy link
Owner

ZeroQI commented Feb 15, 2017

@Dingmatt Excellent news! Thanks for the good work and @KingJ too
Tried the fix but didn't start the agent for me... Agent dev moved along nicely but ep screenshot and title don't update :/
My logging in current dev source show what is updated and not, really look nice

2017-02-15 18:30:15,661 (-beee4c0/common         /UpdateMeta        / 345) INFO     -------------------------------------------------------------------------------------------------------------------------------------------------------------
2017-02-15 18:30:15,662 (-beee4c0/common         /UpdateMeta        / 346) INFO     Plex.UpdateMeta() - Metadata Sources with fields
2017-02-15 18:30:15,662 (-beee4c0/common         /UpdateMeta        / 348) INFO     - AniDB                (  7): rating                 (  1) | genres                 ( 22) | roles                  ( 15) | originally_available_at (  1) | title                  (  1) | summary                (  1) | seasons                (  2) | posters                (  1)
2017-02-15 18:30:15,663 (-beee4c0/common         /UpdateMeta        / 360) INFO       - Episodes           ( 18): duration               ( 18) | rating                 ( 18) | title                  ( 18) | originally_available_at ( 13)
2017-02-15 18:30:15,663 (-beee4c0/common         /UpdateMeta        / 348) INFO     - MyAnimeList          (  0): posters                (  1)
2017-02-15 18:30:15,664 (-beee4c0/common         /UpdateMeta        / 348) INFO     - TheTVDB              ( 10): rating                 (  1) | genres                 (  2) | art                    ( 11) | originally_available_at (  1) | content_rating         (  1) | summary                (  1) | posters                ( 10) | episode_mapping        (  0) | studio                 (  1) | seasons                (  2) | title                  (  1)
2017-02-15 18:30:15,664 (-beee4c0/common         /UpdateMeta        / 359) INFO       - Seasons            (  2): posters                (  2)
2017-02-15 18:30:15,665 (-beee4c0/common         /UpdateMeta        / 360) INFO       - Episodes           ( 19): rating                 ( 18) | summary                ( 18) | title                  ( 19) | thumbs                 ( 18) | originally_available_at ( 15)
2017-02-15 18:30:15,665 (-beee4c0/common         /UpdateMeta        / 361) INFO     -------------------------------------------------------------------------------------------------------------------------------------------------------------
2017-02-15 18:30:15,665 (-beee4c0/common         /UpdateMeta        / 363) INFO     Plex.UpdateMeta() - Metadata Fields
2017-02-15 18:30:15,666 (-beee4c0/common         /UpdateMetaField   / 338) INFO     [=] rating                   Type: float                 Source: AniDB        Value: '4.58'
2017-02-15 18:30:15,666 (-beee4c0/common         /UpdateMetaField   / 338) INFO     [=] genres                   Type: list                  Source: AniDB        Value: '['Small breasts', 'Magical girl', 'Present', 'Fantasy world', 'Earth', 'Absurdist humour', 'Slapstick', 'Parody', 'Japan', 'Urination', 'Large breasts', 'Nudity', 'Shoujo ai', 'Plot continuity', 'Pantsu', 'Asia', 'Parallel universe', 'Science fiction', 'Comedy', 'Ecchi', 'Romance', 'Harem']'
2017-02-15 18:30:15,667 (-beee4c0/common         /UpdateMetaField   / 338) INFO     [=] content_rating           Type: str                   Source: TheTVDB      Value: 'TV-MA'
2017-02-15 18:30:15,667 (-beee4c0/common         /UpdateMetaField   / 338) INFO     [=] art                      Type: dict                  Source: TheTVDB      Value: '{'http://thetvdb.com/banners/fanart/original/103291-5.jpg': ('TVDB/fanart/original/103291-5.jpg', 9, None), 'http://thetvdb.com/banners/fanart/original/103291-7.jpg': ('TVDB/fanart/original/103291-7.jpg', 12, None), 'http://thetvdb.com/banners/fanart/original/103291-11.jpg': ('TVDB/fanart/original/103291-11.jpg', 10, None), 'http://thetvdb.com/banners/fanart/original/103291-10.jpg': ('TVDB/fanart/original/103291-10.jpg', 3, None), 'http://thetvdb.com/banners/fanart/original/103291-1.jpg': ('TVDB/fanart/original/103291-1.jpg', 4, None), 'http://thetvdb.com/banners/fanart/original/103291-3.jpg': ('TVDB/fanart/original/103291-3.jpg', 7, None), 'http://thetvdb.com/banners/fanart/original/103291-2.jpg': ('TVDB/fanart/original/103291-2.jpg', 8, None), 'http://thetvdb.com/banners/fanart/original/103291-9.jpg': ('TVDB/fanart/original/103291-9.jpg', 2, None), 'http://thetvdb.com/banners/fanart/original/103291-4.jpg': ('TVDB/fanart/original/103291-4.jpg', 6, None), 'http://thetvdb.com/banners/fanart/original/103291-6.jpg': ('TVDB/fanart/original/103291-6.jpg', 11, None), 'http://thetvdb.com/banners/fanart/original/103291-8.jpg': ('TVDB/fanart/original/103291-8.jpg', 5, None)}'
2017-02-15 18:30:15,669 (-beee4c0/common         /UpdateMetaField   / 338) INFO     [=] roles                    Type: list                  Source: AniDB        Value: '[{'photo': 'http://img7.anidb.net/pics/anime/17296.jpg', 'role': 'Plug Cryostat', 'name': 'Fukuhara Kaori'}, {'photo': 'http://img7.anidb.net/pics/anime/17475.jpg', 'role': 'Arrester Blanket', 'name': 'Takagaki Ayahi'}, {'photo': 'http://img7.anidb.net/pics/anime/23381.jpg', 'role': 'Oumi Hakone', 'name': 'Miyazaki Ui'}, {'photo': 'http://img7.anidb.net/pics/anime/148709.jpg', 'role': 'Tomonaga Iono', 'name': 'Yahagi Sayuri'}, {'photo': 'http://img7.anidb.net/pics/anime/19811.jpg', 'role': 'Oumi Sento', 'name': 'Takahashi Hiroki'}, {'photo': 'http://img7.anidb.net/pics/anime/177060.jpg', 'role': 'Rona Elmo', 'name': 'Hirano Aya'}, {'photo': 'http://img7.anidb.net/pics/anime/24704.jpg', 'role': 'Pulse Trans', 'name': 'Tomizawa Michie'}, {'photo': 'http://img7.anidb.net/pics/anime/151677.jpg', 'role': 'Kuran Shant', 'name': 'Shimizu Ai'}, {'photo': 'http://img7.anidb.net/pics/anime/184301.jpg', 'role': 'Reika Galvini', 'name': 'Kawasumi Ayako'}, {'photo': 'http://img7.anidb.net/pics/anime/21776.jpg', 'role': 'Rinko', 'name': 'Gibu Yuuko'}, {'photo': 'http://img7.anidb.net/pics/anime/28754.jpg', 'role': 'Sokko', 'name': 'Kinoshita Sayaka'}, {'photo': 'http://img7.anidb.net/pics/anime/45925.jpg', 'role': 'Taako', 'name': 'Shindou Kei'}, {'photo': 'http://img7.anidb.net/pics/anime/151657.jpg', 'role': 'Maako', 'name': 'Ishikawa Ayano'}, {'photo': 'http://img7.anidb.net/pics/anime/170553.jpg', 'role': 'Chiiko', 'name': 'Kaneda Tomoko'}, {'photo': 'http://img7.anidb.net/pics/anime/58661.jpg', 'role': 'Tenchou', 'name': 'Mitsuishi Kotono'}]'
2017-02-15 18:30:15,669 (-beee4c0/common         /UpdateMetaField   / 338) INFO     [=] title                    Type: str                   Source: AniDB        Value: 'Fight Ippatsu! Juuden-chan!!'
2017-02-15 18:30:15,670 (-beee4c0/common         /UpdateMetaField   / 338) INFO     [=] summary                  Type: unicode               Source: AniDB        Value: '* Based on a seinen manga by Ditama Bow, serialised in Comic Gum. Juuden-chan is..'
2017-02-15 18:30:15,670 (-beee4c0/common         /UpdateMetaField   / 338) INFO     [=] posters                  Type: dict                  Source: TheTVDB      Value: '{'http://thetvdb.com/banners/posters/103291-7.jpg': ('TVDB/posters/103291-7.jpg', 9, None), 'http://thetvdb.com/banners/posters/103291-5.jpg': ('TVDB/posters/103291-5.jpg', 6, None), 'http://thetvdb.com/banners/posters/103291-9.jpg': ('TVDB/posters/103291-9.jpg', 2, None), 'http://thetvdb.com/banners/posters/103291-10.jpg': ('TVDB/posters/103291-10.jpg', 1, None), 'http://thetvdb.com/banners/posters/103291-2.jpg': ('TVDB/posters/103291-2.jpg', 8, None), 'http://thetvdb.com/banners/posters/103291-6.jpg': ('TVDB/posters/103291-6.jpg', 4, None), 'http://thetvdb.com/banners/posters/103291-4.jpg': ('TVDB/posters/103291-4.jpg', 5, None), 'http://thetvdb.com/banners/posters/103291-8.jpg': ('TVDB/posters/103291-8.jpg', 11, None), 'http://thetvdb.com/banners/posters/103291-3.jpg': ('TVDB/posters/103291-3.jpg', 7, None), 'http://thetvdb.com/banners/posters/103291-1.jpg': ('TVDB/posters/103291-1.jpg', 10, None)}'
2017-02-15 18:30:15,671 (-beee4c0/common         /UpdateMetaField   / 338) INFO     [=] posters                  Type: dict                  Source: MyAnimeList  Value: '{'https://myanimelist.cdn-dena.com/images/anime/12/19373l.jpg': ('MyAnimeList/images/anime/12/19373l.jpg', 50, None)}'
2017-02-15 18:30:15,671 (-beee4c0/common         /UpdateMetaField   / 338) INFO     [=] posters                  Type: dict                  Source: AniDB        Value: '{'http://img7.anidb.net/pics/anime/22451.jpg': ('AniDB/22451.jpg', 99, None)}'
2017-02-15 18:30:15,672 (-beee4c0/common         /UpdateMetaField   / 338) INFO     [=] studio                   Type: str                   Source: TheTVDB      Value: 'Tokyo MX'
2017-02-15 18:30:15,672 (-beee4c0/common         /UpdateMetaField   / 338) INFO     [=] originally_available_at  Type: date                  Source: AniDB        Value: '2009-06-25'
2017-02-15 18:30:15,673 (-beee4c0/common         /UpdateMeta        / 379) INFO     metadata.seasons[ 0].
2017-02-15 18:30:15,673 (-beee4c0/common         /UpdateMetaField   / 338) INFO     [=] posters                  Type: dict                  Source: TheTVDB      Value: '{'http://thetvdb.com/banners/seasons/103291-0-2.jpg': ('TVDB/seasons/103291-0-2.jpg', 26, None), 'http://thetvdb.com/banners/seasons/103291-0-3.jpg': ('TVDB/seasons/103291-0-3.jpg', 23, None), 'http://thetvdb.com/banners/seasons/103291-0.jpg': ('TVDB/seasons/103291-0.jpg', 27, None)}'
2017-02-15 18:30:15,674 (-beee4c0/common         /UpdateMeta        / 390) INFO     metadata.seasons[ 0].episodes[  1].
2017-02-15 18:30:15,675 (-beee4c0/common         /UpdateMetaField   / 338) INFO     [=] rating                   Type: float                 Source: AniDB        Value: '8.66'
2017-02-15 18:30:15,675 (-beee4c0/common         /UpdateMetaField   / 338) INFO     [1] originally_available_at  Type: date                  Source: TheTVDB      Value: '2009-12-18'
2017-02-15 18:30:15,676 (-beee4c0/common         /UpdateMetaField   / 338) INFO     [=] title                    Type: str                   Source: AniDB        Value: 'ELeC TriCk Music Video'
2017-02-15 18:30:15,676 (-beee4c0/common         /UpdateMetaField   / 338) INFO     [=] summary                  Type: unicode               Source: TheTVDB      Value: 'Whilst the JÅ«den-chan relax at an onsen, Plug gets annoyed at people comparing h..'
2017-02-15 18:30:15,677 (-beee4c0/common         /UpdateMetaField   / 338) INFO     [=] thumbs                   Type: dict                  Source: TheTVDB      Value: '{'http://thetvdb.com/banners/episodes/103291/1499391.jpg': ('TVDB/episodes/1499391.jpg', 1, None)}'

What do you think ?
It is much easier to see which source add what and which fields are filled or not...

@Dingmatt
Copy link
Collaborator Author

@ZeroQI Yep that'll make things easier to diagnose.

@xnaas I'm just happy to help out.

@ZeroQI
Copy link
Owner

ZeroQI commented Feb 27, 2017

can somebody with plex pass test new beta release?

@KingJ
Copy link
Collaborator

KingJ commented Feb 27, 2017

I'll be able to test the beta out later tonight, i'll report back on the other bug thread with the results.

@ZeroQI
Copy link
Owner

ZeroQI commented Feb 27, 2017

👍

@sven-7
Copy link
Contributor

sven-7 commented Feb 27, 2017

The testing I did in #102 was on plex pass for 1.4.x. No longer having issues with posters.

@ZeroQI ZeroQI closed this as completed Mar 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants