Skip to content

Remove requirement for episode number when guessing#1165

Draft
ptrcnull wants to merge 1 commit into
mainfrom
no-episode-check
Draft

Remove requirement for episode number when guessing#1165
ptrcnull wants to merge 1 commit into
mainfrom
no-episode-check

Conversation

@ptrcnull

@ptrcnull ptrcnull commented Aug 2, 2024

Copy link
Copy Markdown
Collaborator

with some types of episodes, such as extras, the episode number will be missing, but the subtitle download succeeds with the check removed

with some types of episodes, such as extras, the episode number
will be missing, but the subtitle download succeeds with the check removed
@ptrcnull ptrcnull requested a review from getzze August 2, 2024 14:28
@github-actions

github-actions Bot commented Aug 2, 2024

Copy link
Copy Markdown
Contributor

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  subliminal
  video.py
Project Total  

This report was generated by python-coverage-comment-action

@ptrcnull ptrcnull marked this pull request as draft August 2, 2024 14:32
@ptrcnull

ptrcnull commented Aug 2, 2024

Copy link
Copy Markdown
Collaborator Author

ah, actually, sometimes it just pulls the first episode of the season...

@getzze

getzze commented Aug 2, 2024

Copy link
Copy Markdown
Collaborator

Fixes #692

And it would be great if this PR could also tackle #678

They don't look easy to fix though...

@getzze

getzze commented Sep 11, 2024

Copy link
Copy Markdown
Collaborator

Are you still working on this?
I just realized it's a duplicate of: #702

@ptrcnull

Copy link
Copy Markdown
Collaborator Author

to be fair, the other MR is flawed in the same way as mine is

@ptrcnull

ptrcnull commented Sep 11, 2024

Copy link
Copy Markdown
Collaborator Author

but to answer your question, i forgot about it, but at some point i'd like to either fix it myself, or see it fixed ^^

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

Successfully merging this pull request may close these issues.

2 participants