Open
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a client-side full-text search experience for seminar PDFs and YouTube transcript captions, including per-slide and timestamp deep links.
Changes:
- Introduces a new seminar search UI, styling, and JS implementation (boolean queries, ranking, snippet highlighting).
- Adds Python builders to generate/search PDF and YouTube inverted indexes with caching.
- Wires the search script into the MkDocs site and adds repo guidance / large-generated-file diff suppression.
Reviewed changes
Copilot reviewed 11 out of 13 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| template/css/base.css | Adds styling for the seminar search widget and results cards. |
| src/seminar/search.js | Implements the client-side search UI and query evaluation for PDF + YouTube indexes. |
| src/seminar/pdf | Adds a symlink pointer to the seminar PDF directory for build scripts. |
| src/seminar/img | Adds a symlink pointer to the seminar image directory for metadata discovery. |
| src/seminar/build-youtube-index.py | Builds a YouTube transcript inverted index from playlist subtitles with caching/reporting. |
| src/seminar/build-pdf-index.py | Builds a PDF text inverted index using pdftotext with caching. |
| src/seminar/build-all.py | Adds a wrapper to build both indexes with exit codes for automation. |
| src/seminar/README.txt | Documents maintenance workflow, query syntax, and generated file policy. |
| src/seminar.md | Adds the search widget markup to the seminar page. |
| mkdocs.yml | Attempts to load the new search JS via MkDocs configuration. |
| .gitattributes | Suppresses diffs for generated index JSON files and marks them as generated. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
|
So far, I have added a note to the README after testing what happens if you have only an abstract, not a video transcript. |
Contributor
|
README line 72 about ordering: "most recent" doesn't seem to work well with PDFs b/c they have no date? Example search term: llvm
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Codex-assisted implementation of search inside the seminar PDFs and YouTube recording transcripts.
The search results have links to the individual slides and video times when the term is mentioned.
See the new
src/seminardirectory for the scripts that produce the search indices and the JS code that performs the search.