Skip to content

docs: modify readme.md 0102#136

Merged
sairin1202 merged 3 commits intomainfrom
feat/readme_0102
Jan 2, 2026
Merged

docs: modify readme.md 0102#136
sairin1202 merged 3 commits intomainfrom
feat/readme_0102

Conversation

@sairin1202
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings January 2, 2026 07:01
@sairin1202 sairin1202 merged commit f114ee4 into main Jan 2, 2026
4 checks passed
@sairin1202 sairin1202 deleted the feat/readme_0102 branch January 2, 2026 07:01
Copy link
Contributor

Copilot AI left a 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 comprehensively restructures the README.md to improve clarity, organization, and user experience. The documentation has been modernized with better visual hierarchy, clearer feature descriptions, and more streamlined content flow.

Key changes include:

  • Updated Python version requirement from 3.8+ to 3.13+ (consistent with pyproject.toml)
  • Reorganized content with improved section hierarchy and table-based feature descriptions
  • Enhanced Quick Start guide with clearer options for cloud vs self-hosted deployment
  • Expanded Core APIs section with detailed retrieval method comparisons

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

Comment on lines 271 to +274
---

## 🤝 Ecosystem

We're proud to work with amazing organizations:
---
Copy link

Copilot AI Jan 2, 2026

Choose a reason for hiding this comment

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

There are duplicate horizontal rules (---) on lines 271 and 274. One of these should be removed to maintain clean formatting.

Copilot uses AI. Check for mistakes.

**Performance Excellence in Locomo Tests:**
```bash
pip install -e .
Copy link

Copilot AI Jan 2, 2026

Choose a reason for hiding this comment

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

The installation instruction uses "pip install -e ." which installs the package in editable/development mode. For a "Quick Start" section aimed at end users, this should likely be "pip install memu-py" (the published package name from PyPI as shown in the badge) unless the intention is specifically for users to clone the repository and install from source in development mode. Consider clarifying the intended installation method or adding both options with clear labels (e.g., "From PyPI" vs "From Source").

Suggested change
pip install -e .
pip install memu-py
Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants