Polish for public release - #1
Merged
Merged
Conversation
Both were superseded by the consolidated backend/ service; nothing imports either, and git history retains them. patient_profile.json moves to the repo root as the single source of patient identity. vision_service/yolov8n.pt is untracked - ultralytics re-downloads it on first run.
Single FastAPI service merging the two prior backends: perception loop over the vision websocket, /ask and /api/caregiver/analyze endpoints, keyword-overlap retrieval over patient_profile.json, and a mock LLM path (MOCK_LLM=true) so the stack runs with no API key. All Gemini calls use the async client so an API round trip cannot stall the event loop, and response.parsed is guarded at every call site. Includes 55 offline tests.
StaticFiles resolved relative to CWD, so the service only started from inside vision_service/; it is now anchored to __file__. Default bind moves from 0.0.0.0 to 127.0.0.1 since no endpoint is authenticated, including GET /frame (live camera stills) and DELETE /people/{name}. Adds a consent notice above the capture button.
The 2s /latest poll overwrote the caretaker's /ask answer because both wrote into #llmResponse; the patient card now has its own element and signatures are tracked per target. Object labels are built as DOM nodes with textContent rather than interpolated into innerHTML. Adds a consent notice to the registration page.
ruff configured with src/known-first-party so the flat backend imports resolve as first-party rather than third-party. gitignore gains .claude/, **/.DS_Store, **/memory_data/ (caregiver-entered personal data), and *.pt. .env.example lists only the variables the live code reads.
Spaces in the old directory name broke shell commands and made it un-importable. Adds a README covering the Ollama prerequisite and both ports; replaces run_command.txt.
MIT license. DISCLAIMER.md states this is a research demo and not a medical device - the model emits a field named clinical_rationale, so this needs saying plainly. PRIVACY.md documents face-embedding storage as biometric data, where it lives, and how to delete it. README rewritten for a stranger: architecture, quickstart with MOCK_LLM=true as the no-API-key path, and the API contract.
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
1 participant
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.