Fix UI startup crash and Multi-GPU row layout - #2
Open
dxqb wants to merge 2 commits into
Open
Conversation
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
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
marked this pull request as ready for review
June 15, 2026 20:56
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
KeyError: 'save_filename_prefix'inmodules/ui/TrainUI.py(create_backup_tab). A merge from master reintroduced a "Save Filename Prefix" row referencingsave_filename_prefix, a field already removed by the "Split model output destination" work on this branch (replaced byrun_name/run_name_mode/final_output_dir). Removed the stale row.Drafted by Claude