-
Notifications
You must be signed in to change notification settings - Fork 0
Translate README to English and fix invalid badges #6
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
Conversation
- Translate main README.md from Chinese to English - Fix CI badge: change YOUR_USERNAME to BukeLy/rag-api - Add language switcher: English/简体中文 - Backup original Chinese README as README.zh-CN.md - Translate all sections: Introduction, Features, Architecture, Quick Start, API Documentation, Deployment, Multi-tenant Usage, Troubleshooting, Contributing - Translate Mermaid diagrams, tables, code comments, and examples - 100% English translation complete (except language switcher label) Co-authored-by: BukeLy <19304666+BukeLy@users.noreply.github.com>
BukeLy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
修复中文文档的Badge
- Fix CI badge: change YOUR_USERNAME to BukeLy/rag-api - Add language switcher to Chinese README for consistency Co-authored-by: BukeLy <19304666+BukeLy@users.noreply.github.com>
Fixed in commit 067737d. Updated the CI badge URL in README.zh-CN.md from |
There was a problem hiding this comment.
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 successfully translates the README from Chinese to English and fixes invalid CI badges. The main README.md is now in English with the original Chinese version saved as README.zh-CN.md, with bidirectional language switchers added to both files.
Key Changes
- Complete translation of README.md (1,277 lines) from Chinese to English covering all sections including features, architecture diagrams, API documentation, deployment guides, and troubleshooting
- Fixed CI badge placeholder
YOUR_USERNAMEtoBukeLy/rag-apiin both English and Chinese versions - Added language switcher links
[English](README.md) | [简体中文](README.zh-CN.md)at the top of both files
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 10 comments.
| File | Description |
|---|---|
| README.md | Fully translated to English; CI badge fixed; language switcher added; minor spacing/punctuation issues in some sections |
| README.zh-CN.md | New file containing original Chinese content; CI badge fixed; language switcher added |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - `full`: ~120 秒(完整上下文处理) | ||
| **Processing Time Estimate** (20-page PDF example): | ||
| - `off`: ~10 seconds(Markdown only) | ||
| - `selective`: ~30 seconds(5-10 important charts) |
Copilot
AI
Dec 12, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing space after "~30 seconds". Should be "~30 seconds (5-10 important charts)"
| - `selective`: ~30 seconds(5-10 important charts) | |
| - `selective`: ~30 seconds (5-10 important charts) |
| **Processing Time Estimate** (20-page PDF example): | ||
| - `off`: ~10 seconds(Markdown only) | ||
| - `selective`: ~30 seconds(5-10 important charts) | ||
| - `full`: ~120 seconds(complete context processing) |
Copilot
AI
Dec 12, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing space after "~120 seconds". Should be "~120 seconds (complete context processing)"
| - `full`: ~120 seconds(complete context processing) | |
| - `full`: ~120 seconds (complete context processing) |
| | `max_entity_tokens` | int | Entity token limit | `6000` | | ||
|
|
||
| 完整 API 文档访问:http://localhost:8000/docs | ||
| Complete API documentation:http://localhost:8000/docs |
Copilot
AI
Dec 12, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing space after "documentation:". Should be "Complete API documentation: http://localhost:8000/docs"
| Complete API documentation:http://localhost:8000/docs | |
| Complete API documentation: http://localhost:8000/docs |
| ``` | ||
|
|
||
| 详细配置参考 [外部存储部署文档](docs/DEPLOYMENT_EXTERNAL_STORAGE.md)。 | ||
| See [External Storage Deployment Documentation](docs/DEPLOYMENT_EXTERNAL_STORAGE.md)。 |
Copilot
AI
Dec 12, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing period at the end of the sentence. Should be "See External Storage Deployment Documentation."
| See [External Storage Deployment Documentation](docs/DEPLOYMENT_EXTERNAL_STORAGE.md)。 | |
| See [External Storage Deployment Documentation](docs/DEPLOYMENT_EXTERNAL_STORAGE.md). |
| - `chore:` Build/tools | ||
|
|
||
| 详见 [PR 工作流程文档](docs/PR_WORKFLOW.md)。 | ||
| See [PR Workflow Documentation](docs/PR_WORKFLOW.md)。 |
Copilot
AI
Dec 12, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove unnecessary period at the end of the link text. Should be "See PR Workflow Documentation"
| See [PR Workflow Documentation](docs/PR_WORKFLOW.md)。 | |
| See [PR Workflow Documentation](docs/PR_WORKFLOW.md) |
| - 确保 `.env` 中配置了 LLM API(豆包) | ||
| **Check Items**: | ||
| 1. **vision_model_func not configured** | ||
| - Check logs:`vision_model_func not found, fallback to off mode` |
Copilot
AI
Dec 12, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inconsistent capitalization: "Check logs:" should be "Check logs: " with a space before the backticks for consistency.
| - `off`: ~10 seconds(Markdown only) | ||
| - `selective`: ~30 seconds(5-10 important charts) | ||
| - `full`: ~120 seconds(complete context processing) |
Copilot
AI
Dec 12, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing space after "~10 seconds". Should be "~10 seconds (Markdown only)"
| - `off`: ~10 seconds(Markdown only) | |
| - `selective`: ~30 seconds(5-10 important charts) | |
| - `full`: ~120 seconds(complete context processing) | |
| - `off`: ~10 seconds (Markdown only) | |
| - `selective`: ~30 seconds (5-10 important charts) | |
| - `full`: ~120 seconds (complete context processing) |
| "embedding": {...}, // Using Global Config | ||
| "rerank": {...}, // Using Global Config | ||
| "ds_ocr": {...}, // Using tenant config | ||
| "mineru": {...} // Using Global Config |
Copilot
AI
Dec 12, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The translation "Using Global Config" appears twice with different capitalization. For consistency, use "Using global config" (lowercase 'g' and 'c') to match the pattern used for other entries in this diagram.
| "mineru": {...} // Using Global Config | |
| "mineru": {...} // Using global config |
| - 检查日志:`Image file not found: xxx` | ||
| - 可能是 MinerU ZIP 损坏或解压失败 | ||
| 2. **Image file does not exist** | ||
| - Check logs:`Image file not found: xxx` |
Copilot
AI
Dec 12, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing space after "logs:". Should be "Check logs: Image file not found: xxx"
| RAG_VLM_MODE=selective # 降级到 selective | ||
| # Modify .env | ||
| VLM_TIMEOUT=300 # Increase to 5 minutes | ||
| RAG_VLM_MODE=selective # downgrade to selective |
Copilot
AI
Dec 12, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The word "downgrade" should be capitalized at the beginning of the comment. Should be "# Downgrade to selective"
| RAG_VLM_MODE=selective # downgrade to selective | |
| RAG_VLM_MODE=selective # Downgrade to selective |
更新 README 到英文版本并修复无效 Badges
任务清单
完成情况
✅ 完全完成 - README 已100%翻译为英文版本,两个版本的 Badges 均已修复
YOUR_USERNAME→BukeLy/rag-apiOriginal prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.