-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Add header to the application #110
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
Open
anands26
wants to merge
12
commits into
https-deeplearning-ai:main
Choose a base branch
from
anands26:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
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
- Make source citations clickable links to lesson videos - search_tools.py: Look up lesson URLs from course catalog - app.py: Update QueryResponse to return structured source data - script.js: Render sources as <a> tags opening in new tabs - Fix PyTorch compatibility for Intel Mac (x86_64) - Pin torch==2.2.2, numpy<2, sentence-transformers==2.6.1 - Use Python 3.11 in run.sh (torch 2.2.2 requires it) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
- Add CourseOutlineTool for retrieving course structure with lessons list - Refactor AIGenerator to support up to 2 sequential tool calling rounds - Add /api/session/new endpoint for creating fresh chat sessions - Add "New Chat" button to frontend for starting fresh conversations - Improve source links display with better styling and hover effects Co-Authored-By: Claude Opus 4.5 <[email protected]>
- Add pytest, pytest-asyncio, and httpx test dependencies - Configure pytest.ini_options in pyproject.toml for cleaner test runs - Create conftest.py with shared fixtures for mocking RAGSystem - Add test_api.py with 15 tests covering all API endpoints - Pin Python to 3.11-3.12 for torch compatibility Co-Authored-By: Claude Opus 4.5 <[email protected]>
- Add black as dev dependency in pyproject.toml with configuration - Create format.sh script to format Python files with black - Create check.sh script for CI-friendly formatting checks - Format all existing Python files for consistency - Add CLAUDE.md with updated development commands Co-Authored-By: Claude Opus 4.5 <[email protected]>
- Add theme toggle button with sun/moon icons in top-right corner - Implement light theme CSS variables for all UI colors - Add smooth 0.3s transitions for seamless theme switching - Persist theme preference to localStorage - Ensure keyboard accessibility for toggle button Co-Authored-By: Claude Opus 4.5 <[email protected]>
…924679 Add Claude Code GitHub Workflow
- Change "Course Materials Assistant" to "Course Material Assistant" - Add <hr> horizontal line below header as requested in issue #2 Co-authored-by: anands26 <[email protected]>
Add horizontal line to header and fix title
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.
No description provided.