Skip to content

Improving readability of progress bar#2584

Open
erikvansebille wants to merge 3 commits intoParcels-code:mainfrom
erikvansebille:simplifying_time_in_progressbar
Open

Improving readability of progress bar#2584
erikvansebille wants to merge 3 commits intoParcels-code:mainfrom
erikvansebille:simplifying_time_in_progressbar

Conversation

@erikvansebille
Copy link
Copy Markdown
Member

Description

This PR improves the readability of the tqdm progress bar by

  • cleaning up the datetime formatting (removing subsecond information)
  • removing the redundant counter information

Before
Screenshot 2026-04-21 at 09 51 05

After
Screenshot 2026-04-21 at 09 54 36

Note: I'm not sure how to write unit tests for this; and whether that is even necessary

Checklist

  • This PR targets the correct branch (main for normal development, v3-support for v3 support)

AI Disclosure

  • This PR contains AI-generated content.
    • I have tested any AI-generated content in my PR.
    • I take responsibility for any AI-generated content in my PR.
    • Describe how you used it (e.g., by pasting your prompt): I asked Claude how to implement the format changes that I wanted

@erikvansebille erikvansebille changed the title Using float_to_datelike for time in progressbar Improving readability of progress bar Apr 21, 2026
@erikvansebille
Copy link
Copy Markdown
Member Author

Perhaps we should change "Integration time" to "Simulation time" or simply "Time" or "Progress"?

@michaeldenes
Copy link
Copy Markdown
Member

I quite liked the 554400.0 / 2678400.0 like information. If my simulation crashed, I could tell immediately from the log at what timestep to investigate. Could this be a toggle-able run-time option (like the rcParams in matplotlib)?

@erikvansebille
Copy link
Copy Markdown
Member Author

I quite liked the 554400.0 / 2678400.0 like information. If my simulation crashed, I could tell immediately from the log at what timestep to investigate. Could this be a toggle-able run-time option (like the rcParams in matplotlib)?

But can't you use the datetime itself for that? That is in front of the progress bar and is also updated. What does the time counter have that the datetime doesn't?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

2 participants