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

Automatic download subtitles #722

Closed
wants to merge 7 commits into from
Closed

Automatic download subtitles #722

wants to merge 7 commits into from

Conversation

ocordeiro
Copy link

Automatic download subtitles from opensubtitles.
Based on checksum from movie files

captura de tela 2016-07-16 as 8 40 04 pm

@bnjmnt4n
Copy link
Member

This is similar to #677. For now, I don't think WebTorrent Desktop will include this feature in its core. Perhaps after the 1.0 release, we'll look into adding extensions such as these.

@xnakos
Copy link

xnakos commented Jul 28, 2016

@alamcordeiro Just a snack for thought; while the file is being downloaded, the checksum is bogus. So, if someone wanted to download subtitles for a video file that is still being downloaded, using the checksum is not an option. If I remember correctly, using SMPlayer to download subtitles while a file was being downloaded provided no results, so it made me think that this method was used. On the other hand, VLC's extension for downloading subtitles provides two methods for searching. One by checksum and one by filename. I think that the latter plays well too.

@grunjol
Copy link
Contributor

grunjol commented Jul 28, 2016

The opensubtitle hash is based on size + 64bit cheksum of the first and last 64k of the file.
You can check piece availability to ensure hash is not bogus (or maybe force first and last blocks).

@xnakos
Copy link

xnakos commented Jul 28, 2016

@grunjol Thanks for the correction! Here is the source too: https://trac.opensubtitles.org/projects/opensubtitles/wiki/HashSourceCodes (a JavaScript implementation is also contained in the page).

@feross
Copy link
Member

feross commented Jul 30, 2016

Thanks for the PR, but we're not going to add this feature to WebTorrent Desktop, because the legality of OpenSubtitles is not clear to us.

Once there's a plugin system, I suppose you could build this as a plugin. But it's not something that should be a part of WebTorrent Desktop core.

@feross feross closed this Jul 30, 2016
@ocordeiro
Copy link
Author

Okay, thanks

@dev-myatminsoe
Copy link

@alamcordeiro Please develop a plugin.

@codealchemist
Copy link
Contributor

@septemberboy7, I contributed with a plugin system in this PR #1125 .
Once we can move forward with it I'd like to create a plugin with what I did on #677 .
I also think that auto downloading subs in WebTorrent Desktop will be awesome!
We need @feross and @dcposch to review the PR.
I also hope to get some feedback on that too :)

@alamcordeiro, if #1125 gets approved maybe we can work together on the subtitles plugin.
Cheers!

@ocordeiro
Copy link
Author

ocordeiro commented Feb 16, 2017

Excellent. I at disposal

@lock lock bot locked as resolved and limited conversation to collaborators May 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants