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

Allow full search of video transcripts #72

Open
scowalt opened this issue Feb 12, 2015 · 5 comments
Open

Allow full search of video transcripts #72

scowalt opened this issue Feb 12, 2015 · 5 comments

Comments

@scowalt
Copy link
Contributor

scowalt commented Feb 12, 2015

Users should have the ability to do a text search of all of the transcripts of all of the lecture videos. Users should see the results of this search as a list of relevant lessons. Users should be able to click on one of the lessons and (potentially) be taken to the exact part of the video relevant to their search term.

@scowalt
Copy link
Contributor Author

scowalt commented Feb 12, 2015

@neelabhg , could you assign myself and / or @jdtran23 to this issue?

@scowalt
Copy link
Contributor Author

scowalt commented Feb 24, 2015

Mozilla has a vtt.js library which could be helpful in implementing this feature.

There is also the Fuse library for fuzzy searching.

@scowalt
Copy link
Contributor Author

scowalt commented Mar 11, 2015

An isolated (but working) prototype of this exists in this repository.

A reverse-index will be continuously maintained and stored in the sysassets repository, which should speed up searching and allow everything to be more scalable.

I'm unsure about where this search functionality would go in the new interface. Perhaps this discussion should be included in #20.

@jdtran23
Copy link
Contributor

Transcript search now lives as a tab alongside the man pages search tab and the code tab.

Current state is that it plays an example video based on an inputted time. The point is that there's now an environment for the transcript search feature utilizing the auto-complete search bar.

The code is currently in my fork.

@scowalt
Copy link
Contributor Author

scowalt commented Apr 3, 2015

I fixed the issue you're having in your fork

Make sure your slaves for xdomain are set as such:

xdomain.slaves({
    'http://cs-education.github.io': '/sysassets/proxy.html',
    'http://jdtran23.github.io': '/sysassets/proxy.html',
    'http://scowalt.com': '/sysassets/proxy.html'
});

However, you should consolidate all of this to the cs-education sysassets since this issue has been resolved.

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