Skip to content

feat(monitor): add tight column width mode - #897

Open
klei22 wants to merge 1 commit into
ReaLLMASIC:masterfrom
klei22:add_width_cycling
Open

feat(monitor): add tight column width mode#897
klei22 wants to merge 1 commit into
ReaLLMASIC:masterfrom
klei22:add_width_cycling

Conversation

@klei22

@klei22 klei22 commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

This pull request enhances the column width adjustment functionality in run_exploration_monitor.py by introducing a new "tight fit" mode, improving user control over column display. The changes also ensure that the new mode is persisted in the configuration and thoroughly tested.

Column Width Adjustment Improvements:

  • Added a new "tight fit" mode for columns, allowing users to cycle through default, fit-to-data, and tightly-fit-to-data widths using the w key. The UI and help text have been updated to reflect this new behavior. [1] [2] [3] [4]
  • Introduced the tight_fit_columns set to track columns in tight fit mode, and updated the logic in _column_width and _cycle_column_width to handle the new mode. [1] [2]

Configuration Persistence:

  • Modified the configuration save/load logic to persist the tight_fit_columns setting, ensuring user preferences are retained across sessions. [1] [2]

Testing:

  • Added unit tests to verify the behavior of the new width cycling, including correct handling when the column heading is longer than the data and ensuring tight fit uses the largest visible value.

These changes provide a more flexible and user-friendly experience when adjusting column widths in the monitor tool.

@klei22
klei22 requested review from gkielian and a lite review from Copilot August 27, 2026 05:36

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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