Skip to content

Wire dh train command to finetune model and smoke test - #5

Open
Shouryaverma19 wants to merge 2 commits into
qorexdevs:mainfrom
Shouryaverma19:main
Open

Wire dh train command to finetune model and smoke test#5
Shouryaverma19 wants to merge 2 commits into
qorexdevs:mainfrom
Shouryaverma19:main

Conversation

@Shouryaverma19

Copy link
Copy Markdown

What does this change?

Replaced the placeholder dh train subcommand stub inside cli.py with the functional execution pipeline calling run_finetune. Also added examples/finetune_smoke.py for headless CI environment validation.

Why?

Closes #3

Testing

  • pytest -q -m "not slow and not api" passes locally
  • ruff check src tests is clean
  • black --check src tests is clean
  • If this changes numbers in the paper, I re-ran make paper-tables and the diff is intentional

Type of change

  • [x ] Bug fix (non-breaking)
  • New task / model adapter / metric
  • Refactor (no user-visible change)
  • Docs only
  • Breaking change (please describe migration below)

Notes for reviewers

Implement fine-tuning functionality in CLI using configuration.
Implement a smoke test for fine-tuning process with temporary output directory and mock execution.
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.

[bug] dh train is a stub — wire it to deterministic_horizon.training.finetune

1 participant