feat/dl: Add comprehensive demo page with DeBERTa v3 Large integration - #168
Closed
d-ulker wants to merge 252 commits into
Closed
feat/dl: Add comprehensive demo page with DeBERTa v3 Large integration#168d-ulker wants to merge 252 commits into
d-ulker wants to merge 252 commits into
Conversation
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.
🎯 SCOPE DECLARATION
ALLOWED: [Add comprehensive demo page showcasing all three AI models]
FORBIDDEN: [No changes to existing demo pages, no model architecture changes, no API endpoint modifications]
FILES TOUCHED: [2 files - website/comprehensive-demo.html, website/js/comprehensive-demo.js]
TIME ESTIMATE: [2 hours]
�� Description
This PR adds a comprehensive demo page that properly showcases our complete AI platform with the correct model architecture. The demo integrates all three AI models: SAMO Whisper (voice transcription), SAMO T5 (text summarization), and SAMO DeBERTa v3 Large (emotion detection with 28 GoEmotions).
✨ Key Features
Complete AI Pipeline Demo
Correct Model Architecture
duelker/samo-goemotions-deberta-v3-largeProduction-Ready Features
�� Technical Implementation
Files Added
website/comprehensive-demo.html- Complete demo page with modern UIwebsite/js/comprehensive-demo.js- Full API integration and functionalityAPI Integration
/predictfor all models)Testing
tests/integration/test_demo_functionality.pyhttp://localhost:8080/comprehensive-demo.html🎨 UI/UX Improvements
�� Testing
�� Deployment
Once merged, the demo will be available at:
https://uelkerd.github.io/SAMO--DL/comprehensive-demo.htmlhttp://localhost:8080/comprehensive-demo.html�� Impact
�� Code Review Checklist
Related Issues: Updates GitHub Pages demo to reflect self-trained DeBERTa v3 Large model
Breaking Changes: None - this is a new demo page
Dependencies: None - uses existing Cloud Run API
Summary by Sourcery
Add a comprehensive end-to-end demo page orchestrating voice transcription, text summarization, and emotion detection, complete with a modern interactive UI, JS client integration, and integration tests; also improve server-side error logging.
New Features:
comprehensive-demo.htmland supportingcomprehensive-demo.jsto demonstrate the full AI pipeline (voice transcription → summarization → emotion detection) in real timeSAMOAPIClient) that handles API requests for Whisper, T5, and DeBERTa modelsBug Fixes:
secure_api_server.pyto log full exception details and return a generic internal server error messageEnhancements:
Tests:
tests/integration/test_demo_functionality.pyto validate demo-frontend API connectivity, request formats, UI components, and GoEmotions labelsSummary by CodeRabbit
New Features
Documentation
Chores