-
Notifications
You must be signed in to change notification settings - Fork 45
OpenAI RFT #202
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
OpenAI RFT #202
Conversation
9d655eb to
0646635
Compare
| skip_vllm_startup: bool = typer.Option( | ||
| False, | ||
| "--skip-vllm-startup", | ||
| "--skip_vllm_startup", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: CLI flag uses underscores instead of hyphens
The skip_vllm_startup parameter option was changed from --skip-vllm-startup to --skip_vllm_startup. Typer CLI flags should use hyphens (kebab-case), not underscores. This breaks the command-line interface and conflicts with the established convention used elsewhere in the CLI (e.g., --vllm-gpu, --reasoning-effort). Users calling this command will encounter an unrecognized flag error.
Note
Introduces RFT (beta) with new
hud rftCLI (run/status) and docs, plus Claude/OpenAI tool fixes and version bump to 0.4.66.hud rftcommand group withrun(submits training jobs) andstatus(fetches raw status); includes task validation (min 10), non-vision check, staging URL patching, and job submission tosettings.hud_rl_url.docs/beta/index.mdxanddocs/beta/rft.mdx; addreference/cli/rft.mdx; update navigation and CLI overview.hud/agents/claude.py): replaceOmitwithNotGiven; guard toolinput(dict vs object); conditionalbetashandling.hud/tools/computer/openai.py): addCoordinatemodel; type-safe dragpath; method signature override for__call__.--skip_vllm_startupinhud/cli/__init__.py.0.4.66inpyproject.toml,hud/version.py, tests, and docs.Written by Cursor Bugbot for commit c581176. This will update automatically on new commits. Configure here.