OpenAI RFT - #202
Merged
Merged
OpenAI RFT#202
Conversation
jdchawla29
force-pushed
the
j/openai-rft
branch
from
November 20, 2025 01:02
9d655eb to
0646635
Compare
lorenss-m
marked this pull request as ready for review
November 21, 2025 04:25
lorenss-m
had a problem deploying
to
pre-release
November 21, 2025 04:25 — with
GitHub Actions
Failure
| skip_vllm_startup: bool = typer.Option( | ||
| False, | ||
| "--skip-vllm-startup", | ||
| "--skip_vllm_startup", |
There was a problem hiding this comment.
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.
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.
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.