Translate README to English and fix invalid badges#6
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>
- 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.
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) |
There was a problem hiding this comment.
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) |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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)。 |
There was a problem hiding this comment.
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)。 |
There was a problem hiding this comment.
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` |
There was a problem hiding this comment.
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) |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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` |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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.