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

Cant parse only season, multiple season and subs excess with only season #17

Open
jonatangrim opened this issue Aug 10, 2017 · 2 comments

Comments

@jonatangrim
Copy link

Hello,

Is it possible to add season parsing, if episode is not existing?

The.X.Files.S01.DKsubs.1080p.BluRay.HEVC.x265 - Returns:
{'quality': 'BluRay', 'codec': 'x265', 'resolution': '1080p', 'excess': 'HEVC', 'title': 'The X Files S01 DKsubs'}
I'm in the need of this result
{'quality': 'BluRay', 'season': '1', 'codec': 'x265', 'resolution': '1080p', 'excess': ['HEVC', 'DKsubs'], 'title': 'The X Files'}

Is it also possible to add multiple season parsing?
The.X.Files.S01-S03.DKsubs.1080p.BluRay.HEVC.x265 - Returns:
{'quality': 'BluRay', 'codec': 'x265', 'resolution': '1080p', 'excess': 'HEVC', 'title': 'The X Files S01-S03 DKsubs'}
I'm the need of this result
{'quality': 'BluRay', 'season':['1','2','3'] 'codec': 'x265', 'resolution': '1080p', 'excess': ['HEVC', 'DKsubs'], 'title': 'The X Files'}

Also - if episode is not existing "DKsubs" becomas a part of the title and not in excess.

Thanks

@jonatangrim jonatangrim changed the title Cant parse full season name Cant parse only season, multiple season and subs excess with only season Aug 10, 2017
@Rhilip
Copy link

Rhilip commented Aug 11, 2017

Oh,it seems same problem as I put forward at #15
The regax pattern for season in this package is LIKE ('season', '(s?([0-9]{1,2}))[ex]').It always ask e (or x, as the begining of episode)

In my mind, It is easy to parse only season ( with change season pattern to (s\d{1,2})).
But parse S01-S03 to 'season':['1','2','3'] may difficulty.

@roidayan
Copy link
Contributor

I actually work on this and will add pull request when I can for multiple seasons and solve the examples mentioned in this ticket

roidayan added a commit to roidayan/parse-torrent-name that referenced this issue Aug 11, 2017
The support is for range like S01-S09.
Also new key subtitles. first match is DKsubs.
Added tests for multi season and subtitles.

Fixes divijbindlish#17
Signed-off-by: Roi Dayan <[email protected]>
roidayan added a commit to roidayan/parse-torrent-name that referenced this issue Aug 11, 2017
The support is for range like S01-S09.
Also new key subtitles. first match is DKsubs.
Added tests for multi season and subtitles.

Fixes divijbindlish#17
Signed-off-by: Roi Dayan <[email protected]>
roidayan added a commit to roidayan/parse-torrent-name that referenced this issue Aug 13, 2017
The support is for range like S01-S09.
Also new key subtitles. first match is DKsubs.
Added tests for multi season and subtitles.

Fixes divijbindlish#17
Signed-off-by: Roi Dayan <[email protected]>
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

3 participants