docs: fix MiniCPM Desk Pet install guidance#353
Open
YOIMIYA66 wants to merge 2 commits into
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR tightens the
Desktop Pet/桌宠sections in the main MiniCPM README files so they match the current MiniCPM Desk Pet documentation and avoid sending users to stale Clawd-on-Desk-era instructions.Changes made:
Clawd-on-Desk-*-arm64.dmginstaller name with the MiniCPM Desk Pet installer name documented by the Desk Pet repository:MiniCPM Desk Pet-*-arm64.dmg..exeinstaller path alongside the macOS DMG path, matching the Desk Pet README's platform-specific installation instructions.README.mdandREADME-cn.md.Apple Silicon / NVIDIA GPU / CPU pathsphrase to the user-facing release targets documented in MiniCPM Desk Pet: macOS Apple Silicon and Windows x64 with Vulkan support.Persona Adapterslanguage while preserving the implementation detail that these adapters are backed by LoRA adapters.cd MiniCPM-Desk-Pet; the previousgit clone ... && ./go.shcommand would run./go.shfrom the parent directory and fail.#给开发者README anchor with the currentdocs/development.mddeveloper setup page.environment checks, model download, and model warm-up/环境检查、模型下载和模型预热instead of the lower-levelsidecar startupphrasing.Reference Checked
I used the current
OpenBMB/MiniCPM-Desk-Petdocumentation as the source of truth for this cleanup:OpenBMB/MiniCPM-Desk-PetREADME,Installationsection:MiniCPM Desk Pet-*-arm64.dmgfrom Releases..exeinstaller.OpenBMB/MiniCPM-Desk-PetREADME /README.zh-CN.md,System Requirementssection:OpenBMB/MiniCPM-Desk-PetREADME /README.zh-CN.md,First Launchsection:Environment Check -> Model Download -> Model Warm-up -> Ready to Use.OpenBMB/MiniCPM-Desk-PetREADME /README.zh-CN.md, developer notes section:docs/development.md.OpenBMB/MiniCPM-Desk-Petrelease metadata:Reference repository: https://github.com/OpenBMB/MiniCPM-Desk-Pet
Why This Matters
The previous MiniCPM README text mixed current MiniCPM Desk Pet guidance with stale upstream Clawd-on-Desk naming. That can confuse users in two ways:
The developer-run command also needed a small but practical fix. Without changing into the cloned repository first, the command cannot find
./go.sh.The same stale content existed in the Chinese README, so this PR keeps the English and Chinese Desktop Pet sections aligned.
Validation
README.mdorREADME-cn.md:Clawd-on-Desk-*-arm64.dmg#给开发者git clone ... && ./go.shwithoutcd MiniCPM-Desk-PetApple Silicon / NVIDIA GPU / CPUsidecar 启动https://github.com/OpenBMB/MiniCPM-Desk-Pet/releaseshttps://github.com/OpenBMB/MiniCPM-Desk-Pet/blob/main/docs/development.mdhttps://raw.githubusercontent.com/OpenBMB/MiniCPM-Desk-Pet/main/README.mdgit diff --checkfor the documentation diff.No code paths or model behavior are changed by this PR.