Skip to content

fix(cli): default LoRA downloads to merged checkpoints#469

Merged
blainekasten merged 1 commit into
mainfrom
cursor/cli-command-failure-analysis-9876
Jul 23, 2026
Merged

fix(cli): default LoRA downloads to merged checkpoints#469
blainekasten merged 1 commit into
mainfrom
cursor/cli-command-failure-analysis-9876

Conversation

@cursor

@cursor cursor Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • default omitted LoRA checkpoint types to merged, matching the CLI contract
  • preserve explicit merged, adapter, and default behavior
  • add regression coverage for downloading a LoRA job without --checkpoint-type

Context

The daily CLI failure export contained Invalid checkpoint type for LoRATrainingType: None. Main still reproduced the bug, and no existing PR covered it.

Tests

  • uv run pytest -n 0 tests/cli/test_fine_tuning.py::TestFineTuningDownload
  • uv run ruff check src/together/lib/cli/api/fine_tuning/download.py tests/cli/test_fine_tuning.py
  • uv run ruff format --check src/together/lib/cli/api/fine_tuning/download.py tests/cli/test_fine_tuning.py
  • uv run pyright src/together/lib/cli/api/fine_tuning/download.py
Open in Web View Automation 

Co-authored-by: Blaine Kasten <blainekasten@gmail.com>
@broly-code-security-scanner

Copy link
Copy Markdown

Broly Security Scan

Note

Clean scan
No vulnerabilities detected in this PR.

Note

Re-scan this PR anytime with /broly scan — useful after /broly undismiss, or to refresh findings without a new push.

Broly — SAST (zai-org/GLM-5.2) · Secrets · SCA · GH Actions (zizmor) · Containers · SBOM · Powered by Together AI

@blainekasten
blainekasten marked this pull request as ready for review July 23, 2026 14:42
@blainekasten
blainekasten merged commit 854f3b8 into main Jul 23, 2026
12 checks passed
@github-actions github-actions Bot mentioned this pull request Jul 23, 2026
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.

2 participants