Skip to content

cli: add 'grl-snam coef-train' (deployable-nav trainer as a first-class command) - #102

Merged
transfix merged 1 commit into
mainfrom
feat/coef-train-cli-command
Sep 27, 2026
Merged

transfix merged 1 commit into
mainfrom
feat/coef-train-cli-command

Conversation

@transfix

Copy link
Copy Markdown
Contributor

coef_train — the trainer that exports the versioned .cvcnav a pure-C++ host loads via coef_mlp::load (the deployable nav policy, including grip/risk + learned-lam via --rollout bicycle --w-risk --learned-lam) — was only reachable as python -m grl_snam.tools.coef_train. This makes it a first-class command of the installed CLI: grl-snam coef-train.

Implementation: a thin passthrough (nargs=-1, Click -h/--help disabled so they reach argparse) to grl_snam.tools.coef_train:main(argv) — the tool's argparse stays the single source of truth for its ~18 flags (no option-mirror drift). Distinct from the existing grl-snam train (self-supervised SDF → .pt).

Verified:

  • grl-snam --help lists coef-train.
  • grl-snam coef-train --help prints the full flag list (--rollout/--steps/--w-risk/--lam-soft/--learned-lam/--curriculum/--cuda/...).
  • grl-snam coef-train --rollout bicycle --w-risk 0.5 --learned-lam --lam-soft 0.4 --steps 2 --out x.cvcnav trains + writes a 6-in/4-out .cvcnav.

…t-class command

coef_train (the CoefMLP trainer that exports the versioned .cvcnav a pure-C++ host
loads via coef_mlp::load — the DEPLOYABLE nav policy, incl. grip/risk+learned-lam via
--rollout bicycle) was only reachable as 'python -m grl_snam.tools.coef_train'. Wire it
into the grl-snam Click CLI as 'grl-snam coef-train', a thin passthrough to the tool's
argparse (single source of truth for its ~18 flags; -h/--help pass through). Distinct
from 'grl-snam train' (SDF self-supervised -> .pt). Verified: lists in --help, --help
passes through, and a 2-step run trains + writes a 6-in/4-out .cvcnav.
@transfix
transfix merged commit 2adec75 into main Sep 27, 2026
3 checks passed
@transfix
transfix deleted the feat/coef-train-cli-command branch September 27, 2026 06:03
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