Skip to content

fix: training config #609

Open
sdas-neuraco wants to merge 1 commit into
mainfrom
feat/config-appending-training
Open

fix: training config #609
sdas-neuraco wants to merge 1 commit into
mainfrom
feat/config-appending-training

Conversation

@sdas-neuraco
Copy link
Copy Markdown
Contributor

@sdas-neuraco sdas-neuraco commented May 11, 2026

Features

  • Added support for composing user training YAML files via Hydra's optional user config group, so training arguments can live in reusable config files instead of long CLI commands.
  • Introduced algorithm_name as packaged-algorithm shorthand, resolving it to the matching bundled algorithm config while preserving support for cloud algorithm_id workflows.
  • Renamed local run naming config from run_name / run_name_auto_increment to training_name / training_name_auto_increment, including output directory resolution and documentation updates.
  • Added training config normalization utilities in neuracore/ml/utils/training_config.py for default merging, algorithm resolution, dataset lookup, data type shorthand expansion, robot name/id conversion, and validation.
  • Added get_robot_name_from_id to support converting resolved robot IDs back to user-facing robot names for saved training metadata.
  • Moved all validation and completion checks outside of train.py

@sdas-neuraco sdas-neuraco added the version:minor non-breaking feature updates, new functions or endpoints label May 11, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Consider updating changelogs/pending-changelog.md with a summary of this change for the release notes. This is optional and non-blocking.

@sdas-neuraco sdas-neuraco marked this pull request as ready for review May 11, 2026 16:45
@sdas-neuraco sdas-neuraco self-assigned this May 11, 2026
Copy link
Copy Markdown
Contributor

@ypang-neuraco ypang-neuraco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM \w comments

Comment thread neuracore/ml/train.py Outdated
Comment thread neuracore/ml/train.py
Comment thread neuracore/ml/utils/training_config.py
Comment thread neuracore/ml/utils/training_config.py Outdated
Comment thread neuracore/core/robot.py Outdated
@sdas-neuraco sdas-neuraco changed the title fix training config pre-commit issues fix training config May 13, 2026
@sdas-neuraco sdas-neuraco changed the title fix training config fix: training config May 13, 2026
@sdas-neuraco sdas-neuraco force-pushed the feat/config-appending-training branch from 746a9a7 to 1dfbdec Compare May 13, 2026 16:13
@sdas-neuraco sdas-neuraco force-pushed the feat/config-appending-training branch from 2d37d78 to 683da64 Compare May 14, 2026 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

version:minor non-breaking feature updates, new functions or endpoints

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants