Skip to content

Fix UI startup crash and Multi-GPU row layout - #2

Open
dxqb wants to merge 2 commits into
O-J1:input-tooltips-and-cofrom
dxqb:fix/save-filename-prefix-1300
Open

Fix UI startup crash and Multi-GPU row layout#2
dxqb wants to merge 2 commits into
O-J1:input-tooltips-and-cofrom
dxqb:fix/save-filename-prefix-1300

Conversation

@dxqb

@dxqb dxqb commented Jun 15, 2026

Copy link
Copy Markdown

Summary

  • Fixes a startup crash: KeyError: 'save_filename_prefix' in modules/ui/TrainUI.py (create_backup_tab). A merge from master reintroduced a "Save Filename Prefix" row referencing save_filename_prefix, a field already removed by the "Split model output destination" work on this branch (replaced by run_name / run_name_mode / final_output_dir). Removed the stale row.
  • Fixes the "Device Indexes" row in the General tab: a previous merge left it on its own row (13, cols 2-3) while "Multi-GPU" stayed on row 12, leaving mismatched empty cells in both rows. Restored pairing them on the same row (12), matching upstream.

Drafted by Claude

A merge from master reintroduced this row referencing the
save_filename_prefix field, which was removed by the Split model
output destination changes already on this branch (replaced by
run_name/run_name_mode/final_output_dir). The stale field reference
caused a KeyError on startup.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@dxqb
dxqb marked this pull request as draft June 15, 2026 19:56
A previous merge left Device Indexes on its own row (13, cols 2-3)
while Multi-GPU stayed on row 12, leaving mismatched empty cells in
both rows. Upstream pairs them on the same row; restore that.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@dxqb dxqb changed the title Remove stray Save Filename Prefix UI entry Fix UI startup crash and Multi-GPU row layout Jun 15, 2026
@dxqb
dxqb marked this pull request as ready for review June 15, 2026 20: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