You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
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.
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
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, sincevideo.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.
The text was updated successfully, but these errors were encountered: