Skip to content

fix: correct hf download command format in RKLLM docs#1579

Closed
tangzz-radxa wants to merge 2 commits intoradxa-docs:mainfrom
tangzz-radxa:docs/fix-issue-1563-hf-download
Closed

fix: correct hf download command format in RKLLM docs#1579
tangzz-radxa wants to merge 2 commits intoradxa-docs:mainfrom
tangzz-radxa:docs/fix-issue-1563-hf-download

Conversation

@tangzz-radxa
Copy link
Copy Markdown
Contributor

Summary

This PR fixes the hf download command format in two RKLLM documentation files:

  • common/dev/_rkllm_qwen2_vl.mdx (Chinese and English versions)
  • common/dev/_rkllm-deepseek-r1.mdx (Chinese and English versions)

Changes

The hf download command expects the format owner/repo, not a full URL. This PR removes the https://huggingface.co/ prefix from the commands:

Before:

hf download https://huggingface.co/Qwen/Qwen2-VL-2B-Instruct --local-dir ./Qwen2-VL-2B-Instruct

After:

hf download Qwen/Qwen2-VL-2B-Instruct --local-dir ./Qwen2-VL-2B-Instruct

Related Issues

Testing

The fix is straightforward and follows the official huggingface_hub CLI documentation. The hf download command expects repository identifiers in the owner/repo format, not full URLs.

Bilingual Updates

Both Chinese (docs/) and English (i18n/en/) versions have been updated to maintain consistency.

- Fix hf download command in _rkllm_qwen2_vl.mdx (remove https://huggingface.co/ prefix)
- Fix hf download command in _rkllm-deepseek-r1.mdx (remove https://huggingface.co/ prefix)
- Both fixes address the same pattern: hf download expects 'owner/repo' format, not full URL

Related issues: radxa-docs#1563, radxa-docs#1562
- Fix hf download command in English version of _rkllm_qwen2_vl.mdx (remove https://huggingface.co/ prefix)
- Fix hf download command in English version of _rkllm-deepseek-r1.mdx (remove https://huggingface.co/ prefix)
- Both fixes address the same pattern: hf download expects 'owner/repo' format, not full URL

Related issues: radxa-docs#1563, radxa-docs#1562
@zifeng-radxa
Copy link
Copy Markdown
Contributor

fix in #1565

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