Skip to content

(#2567) fix htmx renderer for checkpoint selection#2568

Merged
bghira merged 2 commits intomainfrom
bugfix/2567
Feb 2, 2026
Merged

(#2567) fix htmx renderer for checkpoint selection#2568
bghira merged 2 commits intomainfrom
bugfix/2567

Conversation

@bghira
Copy link
Owner

@bghira bghira commented Feb 2, 2026

This pull request enhances the handling and population of options for the resume_from_checkpoint field in the form, ensuring that available checkpoints in the specified output directory are accurately listed and selectable. It also improves the robustness of the front-end logic for detecting the output directory input and adds a test to verify the new behavior.

Backend improvements to checkpoint option handling:

  • Added logic to the FieldService to dynamically populate the resume_from_checkpoint field's options with available checkpoints from the specified output directory, ensuring deduplication and inclusion of the current value if not already present.
  • Imported CheckpointManager to support checkpoint discovery in the backend service.

Frontend robustness:

  • Updated JavaScript in form_field.html to reliably detect the output directory input field by checking multiple possible selectors, improving compatibility with different form layouts. [1] [2]

Testing:

  • Added a unit test (test_field_service_resume_from_checkpoint.py) to verify that local checkpoints are correctly included in the options for the resume_from_checkpoint field.

@bghira bghira merged commit 0da5967 into main Feb 2, 2026
2 checks passed
@bghira bghira deleted the bugfix/2567 branch February 2, 2026 16:56
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