umbrellaplug / umbrellaplug.github.io

Umbrella Kodi Addon Official
111 stars 17 forks source link

[FEATURE] Season "offset" #373

Open CnC-ode opened 1 week ago

CnC-ode commented 1 week ago

Feature

Hello,

I have an issue with a show I casually watch. The show is called "Whose Line Is It Anyway? (US)" and it has 2 TMDB ids:

I tried to use the "Require Year in source for known Reboot shows" option but that didn't work.

As I was digging down the problem, I found out that, because the original show (61018) had 8 seasons, the sources for the reboot (64978) have a season "offset" of 8. For example, reboot season 1 will have sources with "S09EXX" instead of "S01EXX"...

I managed to "solve" the problem with some strategically placed hardcoded lines on sources.py xD

Even though this is a niche issue, I though I'd share it because I believe other shows might suffer from the same problem and this is not an elegant solution at all. Pretty sure I will need to add the lines after each addon update.

Maybe you could consider implementing a feature to manage this "offset" based on TMDB id.

Cheers!

umbrellaplug commented 1 week ago

Yeah this is where is gets tough. There's exceptions for different shows / movies written into different parts of umbrella. This is likely going to be another one of those situations. There's ton's of issues with different shows and their metadata not matching. Dr Who is a great example. I can't really write every possible scenario for every show but I will look into what I can do for this. It's just always jacked up metadata.

CnC-ode commented 1 week ago

Thanks for the quick reply.

Of course, I understand that there must be a ton of different issues like this and I truly congratulate you for taking the time and try to tackle them one by one! I actually did stumble upon some hardcoded stuff like this for other shows in your code. I just thought to share this one as other people may be having the same problem with this show or another that falls into the same scenario.

While I was digging up the problem I had this idea where instead of tackling specific scenarios, a feature could be developed in order to allow custom rules for the end user to manage themselves. For example, in this case it would be "as simple as" a pair of tmdbid & "season offset". Of course there would be much complicated scenarios than this which could not be easily handled with custom rules, but at least you wouldn't need to hardcode stuff every time someone stumbled upon another one.

That said, I managed to find this solution for this specific scenario and I will happily live with it xD

Nevertheless, I'm just giving my 2 cents in the spirit of sharing and contributing for a addon I truly love.

Kudos!