Skip to content

Documentation/ai update docs - #34

Merged
LynnDelaere merged 2 commits into
mainfrom
documentation/ai-update-docs
Dec 11, 2025
Merged

LynnDelaere merged 2 commits into
mainfrom
documentation/ai-update-docs

Conversation

@LynnDelaere

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings December 11, 2025 10:42
@LynnDelaere
LynnDelaere merged commit 026d28a into main Dec 11, 2025
5 checks passed
@LynnDelaere
LynnDelaere deleted the documentation/ai-update-docs branch December 11, 2025 10:43

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the AI architecture documentation to reflect current implementation details, including corrected API endpoints, updated library versions, revised model architecture parameters, and additional context about the ASL model's limitations.

Key changes:

  • Updated API endpoint paths to match actual implementation (e.g., /alphabet/asl/predict instead of /asl/predict)
  • Corrected technology versions to match current dependencies (PyTorch 2.7.0, MediaPipe 0.10.21, etc.)
  • Updated LSTM model dropout rates from 0.4 to 0.2 to reflect actual architecture
Comments suppressed due to low confidence (1)

docs/Architecture/AI-Architecture.md:18

  • The referenced file AI-Packages.md does not exist in the repository. This creates a broken link that will result in a 404 error when users try to follow the reference. Either create the docs/Architecture/AI-Packages.md file with the promised content about the smart_gestures package, model lifecycle, and CLI options, or remove this reference until the file is ready.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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

Successfully merging this pull request may close these issues.

2 participants