Skip to content

Docint documentation update - #245

Merged
AndreaChazarreta merged 3 commits into
developfrom
docint/readme-update
Oct 2, 2025
Merged

AndreaChazarreta merged 3 commits into
developfrom
docint/readme-update

Conversation

@AndreaChazarreta

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings October 2, 2025 16:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

Updates the document-intelligence project documentation with comprehensive technology stack details and improved setup instructions.

  • Documents currently used frameworks, technologies, and their experiences (good/bad)
  • Replaces Docker commands with docker-compose setup
  • Updates environment variable configuration with proper .env.example file

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
document-intelligence/README.md Added detailed technology stack documentation and updated setup instructions
document-intelligence/.env.example Created proper environment variable template file
document-intelligence/exampleEnv Removed old environment file format

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

- **Backend:** FastAPI (Python)
- **Dependency management:** Poetry
- **Containerization:** Docker (for containerized development with hot-reload)
- **LLM for text extraction:** Gemma 27 B (used; Gemma 4 B & 12 B were not good)

Copilot AI Oct 2, 2025

Copy link

Choose a reason for hiding this comment

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

The model size should be written as '27B' instead of '27 B' to follow standard naming conventions for language models.

Copilot uses AI. Check for mistakes.
- **OpenAPI code generator:** Worked well for generating client code.

### Bad / Limitations
- **Gemma 4 B & 12 B:** Poor performance for text extraction.

Copilot AI Oct 2, 2025

Copy link

Choose a reason for hiding this comment

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

The model sizes should be written as '4B & 12B' instead of '4 B & 12 B' to follow standard naming conventions for language models.

Copilot uses AI. Check for mistakes.
Comment on lines +7 to +8
# Uncomment next line to enable the debug mode
# ORPHEUS_DEBUG=1

Copilot AI Oct 2, 2025

Copy link

Choose a reason for hiding this comment

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

The environment variable name 'ORPHEUS_DEBUG' is inconsistent with 'ORPHEUS_VERBOSE' mentioned in the removed exampleEnv file. Consider using consistent naming across the project.

Suggested change
# Uncomment next line to enable the debug mode
# ORPHEUS_DEBUG=1
# Uncomment next line to enable the debug/verbose mode
# ORPHEUS_VERBOSE=1

Copilot uses AI. Check for mistakes.
@AndreaChazarreta
AndreaChazarreta merged commit c75e178 into develop Oct 2, 2025
2 checks passed
@AndreaChazarreta
AndreaChazarreta deleted the docint/readme-update branch October 2, 2025 16:05
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