subliminal icon indicating copy to clipboard operation
subliminal copied to clipboard

Season match inconsistency

Open ratoaq2 opened this issue 8 years ago • 2 comments

For the latest version: 2.0.3

There's a season match inconsistency when release names have no season (I'll provide real examples later today).

Given a video file as bellow: Show.Name.E05.720p.x264.mkv

The created video object will have season 1, since guessit will not return season, but this code will default it to 1.

Then when the subtitles (e.g.: Show.Name.E05.720p.x264,srt) are found (by any provider), some/most of them will use guessit to retrieve all the release information and this code will prevent a season match, since video.season == 1 but 'season' is not in guess

Either the season shouldn't be defaulted to 1 in the video or the compute score should follow the same logic.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

ratoaq2 avatar Jul 27 '16 08:07 ratoaq2

I agree on that. The code you linked to was introduced to handle mini-series that don't have a season number in the filename. Something more clever is needed here.

Diaoul avatar Aug 16 '16 06:08 Diaoul

Just to illustrate and give some real examples, there are series that doesn't follow season/episode numbering on their release names. Not all providers have subtitles for those series. Here are some examples of episodes without season and/or episode numbers:

  • Air by date (guessit: date): documentaries episodes or talk shows Catalyst.2015.11.10.Einstein.360p.LDTV.WEBRIP.[MPup] Eastenders.2016.05.27.HDTV.x264-DEADPOOL
  • Absolute episode numbering (guessit: episode): animes Kiznaiver.E06.SUBFRENCH.720p.HDTV.x264-D4KiD
  • Absolute episode using part (guessit: part): The.Night.Of.Part.8.INTERNAL.720p.HDTV.x264

ratoaq2 avatar Sep 04 '16 05:09 ratoaq2