Skip to content

fix(ui): improve progress bar message readability#64

Merged
jhamon merged 1 commit intomainfrom
fix/progress-message-wording
Feb 3, 2026
Merged

fix(ui): improve progress bar message readability#64
jhamon merged 1 commit intomainfrom
fix/progress-message-wording

Conversation

@jhamon
Copy link
Contributor

@jhamon jhamon commented Feb 3, 2026

Problem

When loading datasets, the progress bar displays grammatically awkward messages like:

  • "Loading documents parquet files"
  • "Loading queries parquet files"

The issue is using plural nouns ("documents", "queries") as adjective modifiers before "parquet files", which reads unnaturally.

Solution

Simplified the progress bar description to just "Loading documents" and "Loading queries". Since all dataset files are stored exclusively as parquet format (this is enforced in the code), mentioning "parquet files" is redundant implementation detail that doesn't help users.

Changes

  • Updated dataset_fsreader.py line 99: changed tqdm description from f"Loading {data_type} parquet files" to f"Loading {data_type}"
  • Results in cleaner, more professional progress messages

Testing

  • ✅ All 190 tests pass (3 skipped as expected)
  • ✅ No breaking changes
  • ✅ Backward compatible - only affects UI text

Example

Before:

Loading documents parquet files: 100%|████████| 2/2 [00:01<00:00,  1.23it/s]

After:

Loading documents: 100%|████████| 2/2 [00:01<00:00,  1.23it/s]

The simpler message is easier to read and sounds more natural while conveying the same information to users.

Made with Cursor


Note

Low Risk
Low risk: only changes the tqdm progress bar description string during dataset parquet loading, with no impact on IO, parsing, or schema validation logic.

Overview
Improves dataset loading UX by simplifying the tqdm progress message in DatasetFSReader._safe_read_from_path from “Loading {data_type} parquet files” to just “Loading {data_type}”, removing redundant/awkward wording while keeping behavior unchanged.

Written by Cursor Bugbot for commit 2131394. This will update automatically on new commits. Configure here.

Replace grammatically awkward "Loading documents parquet files"
with cleaner "Loading documents" message.

Co-authored-by: Cursor <cursoragent@cursor.com>
@jhamon jhamon added enhancement New feature or request agent-pr Pull request created by AI agent labels Feb 3, 2026
@jhamon jhamon merged commit a893158 into main Feb 3, 2026
11 checks passed
@jhamon jhamon deleted the fix/progress-message-wording branch February 3, 2026 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent-pr Pull request created by AI agent enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant