Skip to content

fix(tui): Properly display Status in runs widgets#198

Closed
VictorMorand wants to merge 1 commit into
experimaestro:masterfrom
VictorMorand:master
Closed

fix(tui): Properly display Status in runs widgets#198
VictorMorand wants to merge 1 commit into
experimaestro:masterfrom
VictorMorand:master

Conversation

@VictorMorand
Copy link
Copy Markdown
Contributor

@VictorMorand VictorMorand commented Mar 10, 2026

Just fixed the logic to Read Experiment Status in the runs tui widget

@bpiwowar
Copy link
Copy Markdown
Collaborator

Just to check – where those {run_id}.bak files come from?

@VictorMorand
Copy link
Copy Markdown
Contributor Author

After investigation, The duplication rather came from the "current" symlink that points to an existing folder - duplicating the row

@bpiwowar
Copy link
Copy Markdown
Collaborator

OK, so if this is the case, it should be handled directly rather than having an error-prone conflict resolution?

@VictorMorand
Copy link
Copy Markdown
Contributor Author

ok added a simpler fix - just ignore if row was already added

@bpiwowar
Copy link
Copy Markdown
Collaborator

I think it should be more in get_experiment_runs that we need to change things (rather than in the UI) / just use symbolic link resolution to de-deduplicate (we should have no duplicate run id then / otherwise warning)

@VictorMorand VictorMorand changed the title fix(tui): prevent DuplicateKey error in runs list over SSH fix(tui): Properly display Status in runs widgets Mar 11, 2026
@VictorMorand
Copy link
Copy Markdown
Contributor Author

Apologies, there was already a symlink filter, I didn't see that I was using an older version of xpm on the remote, which caused the issue, there is then only the status display fix left

@bpiwowar bpiwowar closed this Mar 11, 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