Skip to content

修复 --cache_dir 在子命令前使用时无法识别的问题#662

Open
Li2718 wants to merge 1 commit intoztxz16:masterfrom
Li2718:fix-cache-dir-cli
Open

修复 --cache_dir 在子命令前使用时无法识别的问题#662
Li2718 wants to merge 1 commit intoztxz16:masterfrom
Li2718:fix-cache-dir-cli

Conversation

@Li2718
Copy link
Copy Markdown
Contributor

@Li2718 Li2718 commented Apr 21, 2026

问题背景

  • 当前 CLI 只支持把共享参数写在子命令后面,例如 ftllm run Qwen/Qwen2-0.5B-Instruct --cache_dir /mnt
  • 当用户按 ftllm --cache_dir /mnt run Qwen/Qwen2-0.5B-Instruct 这种写法使用时,会报 unrecognized arguments
  • 中文 README 中 --cache_dir 的示例也写成了前置形式,容易直接触发这个问题

修改内容

  • 调整 CLI 参数解析逻辑,使共享参数既可以写在子命令前,也可以写在子命令后
  • 保持原有默认值行为,避免顶层参数被子命令默认值覆盖
  • 修正 README 中 --cache_dir 的示例写法

验证

  • ftllm --cache_dir /mnt run Qwen/Qwen2-0.5B-Instruct
  • ftllm run Qwen/Qwen2-0.5B-Instruct --cache_dir /mnt
  • ftllm config demo.json

以上场景均已本地验证解析正常。

Issue

@Li2718 Li2718 changed the title fix: accept cache_dir before cli subcommands 修复 --cache_dir 在子命令前使用时无法识别的问题 Apr 21, 2026
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.

1 participant