Redshirt 1487 Posted March 2, 2016 Posted March 2, 2016 As the title suggests, I'd like to see TMDB added to the list of sources for episode metadata. Some shows actually have better metadata on TMDB. For example, Black sails has no overview for any season 3 episode on thetvdb, but does on tmdb http://thetvdb.com/?tab=episode&seriesid=262407&seasonid=643716&id=5392170&lid=7 https://www.themoviedb.org/tv/47665-black-sails/season/3 4
Luke 42077 Posted March 2, 2016 Posted March 2, 2016 Yea it's a good idea. Maybe @@softworkz will have interest here.
softworkz 5066 Posted March 2, 2016 Posted March 2, 2016 The problem I see here, is that it may be just the other way round for different series. IMO to make this useful, we would need a way to override the primary provider at the series level.
spootdev 56 Posted March 2, 2016 Posted March 2, 2016 The problem I see here, is that it may be just the other way round for different series. IMO to make this useful, we would need a way to override the primary provider at the series level. That would be handy indeed.
ebr 16169 Posted March 2, 2016 Posted March 2, 2016 Data from different providers is combined already.
softworkz 5066 Posted March 2, 2016 Posted March 2, 2016 Data from different providers is combined already. Not for episode data (at least not last time I looked into it)
softworkz 5066 Posted March 2, 2016 Posted March 2, 2016 Previously, there has been TheTVDB as the only provider for episodes. Then, I added an episode provider for OMDB. But lookup will only fall through from one provider to the next provider of lower priority when the first one didn't find anything. What the OP is referring to is a situation where one provider has data for an episode, but another provider would have better data (more details) for the same episode.
ebr 16169 Posted March 2, 2016 Posted March 2, 2016 Okay, sorry. At one point in time, we combined them. Perhaps that changed.
Redshirt 1487 Posted March 2, 2016 Author Posted March 2, 2016 I also didn't realize they weren't combined. It makes sense since there used to only be one provider for episodes. So to refine my original post I'd like to see episode meta combined as well as TMDB added as a source
Luke 42077 Posted March 2, 2016 Posted March 2, 2016 thats' the reason we have ordering. you put the one you want at the top of the list.
softworkz 5066 Posted March 2, 2016 Posted March 2, 2016 @ebr: I have to apologize. I just debugged and both providers are called and results are merged together then. Though, I could swear that when I developed the OmdbEpisodeProvider, I had to move it to the top of the list because otherwise it wasn't executed...
softworkz 5066 Posted March 3, 2016 Posted March 3, 2016 Good news - I just finished implementing the new provider: https://github.com/MediaBrowser/Emby/pull/1515 By leveraging existing stuff from MovieDbSeriesImageProvider and MovieDbSeasonProvider I could get this done without much effort. As final test I used the exact example described by the OP: Disabled the new provider Added series "Black Sails" alongside some dummy files named as season 3 episodes After a library scan the "Overview" field of the episodes was empty while other metadata was there (TheTvDb just doesn't have overview descriptions for S03-Episodes) Activated the new provider Refreshed metadata via Metadata Manager==> Overview information was added as expected 4
Luke 42077 Posted March 3, 2016 Posted March 3, 2016 Great work. Make sure the new provider is disabled by default and ordered to run last. Basically the same thing you did with omdb.
softworkz 5066 Posted March 3, 2016 Posted March 3, 2016 Great work. Make sure the new provider is disabled by default and ordered to run last. Basically the same thing you did with omdb. I just added the same kind of migration task as I did for the OmdbEpisodeProvider. Regarding query performance I would like to note that the new provider is less critical than the OmdbEpisodeProvider because it caches its results to disk (same like TvdbEpisodeProvider).
Redshirt 1487 Posted March 5, 2016 Author Posted March 5, 2016 Just tested this with today's dev release. Worked like a charm. Thanks.
Luke 42077 Posted March 5, 2016 Posted March 5, 2016 @@softworkz, make sure it supports by date lookup, if the tmdb api supports it
softworkz 5066 Posted March 6, 2016 Posted March 6, 2016 Just tested this with today's dev release. Worked like a charm. Thanks. Great news! Thanks for reporting...
softworkz 5066 Posted March 6, 2016 Posted March 6, 2016 @@softworkz, make sure it supports by date lookup, if the tmdb api supports it I studied the API, but it doesn't seem like they were supporting tv episode queries by date: http://docs.themoviedb.apiary.io/#reference/tv-episodes/tvidseasonseasonnumberepisodeepisodenumberrating
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now