Incorrect/irrelevant remaining time estimate before first update

fsaxen / ParforProgMon

Progress monitor for matlab parfor (parallel) loops with estimation of the remaining time

BSD 3-Clause "New" or "Revised" License

27 stars 8 forks source link

Incorrect/irrelevant remaining time estimate before first update #4

Closed Dev-iL closed 5 years ago

Dev-iL commented 5 years ago

Problem Description

Before the first iteration is completed, the estimated time is "Inf week", this should be changed to something more meaningful like "Waiting", "Waiting for the first iteration to complete", "Estimating ETA..." or something similar.

fsaxen commented 5 years ago

Sorry Dev-iL, i somehow missed this issue somehow. In the latest commit ef4b9ba I don't get any time estimate when there hasn't been a single increment call from any worker. Having no times estimate at the beginning is the default in the progressbar implementation I use and I'd prefer to keep it that way. Do you still get "Inf week" estimates in the latest commit?

Dev-iL commented 5 years ago

I've opened this issue the shortly after the first one (which you marked as fixed), so it might've been lost in the emails or notifications.

In any case, this issue was written before your fix to issue #1, and having tried to reproduce the problem again with the latest version, this issue also appears to be fixed - so I'm closing it.