kapitainsky / RcloneBrowser

Simple cross platform GUI for rclone. Supports macOS, GNU/Linux, BSD family and Windows.
https://github.com/kapitainsky/RcloneBrowser
MIT License
2.29k stars 218 forks source link

ETA field in transfers #112

Open dia3olik opened 4 years ago

dia3olik commented 4 years ago

Another one! Ha!

What about having an ETA field constantly updated and calculated from the remaining time so you exactly know when that transfer would finish (at which hour:minute)?

It's a much more practical way of interact with the user imho.

If it's a question of real estate on the GUI I would gladly with the current remaining time field.

Maybe having an option in the preference to choose one to the other? ;-)

And using current system settings as to use AM/PM or 24h EU standard ;-)

kapitainsky commented 4 years ago

Very cosmetic change, happy to add it when implemented.

ETA is captured from rclone output (Transferred: line):

2020/04/28 08:26:41 INFO  :
Transferred:          240k / 688.389 MBytes, 0%, 1.727 kBytes/s, ETA 4d17h20m47s
Checks:               363 / 363, 100%
Transferred:            0 / 3663, 0%
Elapsed time:      2m18.9s
Transferring:
* RcloneBrowser/release/rclone-browser-1.9.6.1-34bcc1b-windows-64-bit.exe:  0% /15.740M, 18/s, 251h15m7s
* RcloneBrowser/release/rclone-browser-1.9.6.1-34bcc1b-windows-64-bit.zip:  0% /19.323M, 138/s, 40h34m15s
* RcloneBrowser/release/rclone-browser-1.9.6.1-6059c46-windows-64-bit.exe:  0% /14.394M, 49/s, 83h45m42s
* RcloneBrowser/release/rclone-browser-1.9.6.1-6059c46-windows-64-bit.zip:  0% /17.510M, 386/s, 13h9m20s

If you would like to change its format it should be very straightforward. Here is the line of code that has to be changed to update ETA field:

https://github.com/kapitainsky/RcloneBrowser/blob/dabd09d1f07242af69bcb808f3197fe940b46938/src/job_widget.cpp#L113

dia3olik commented 4 years ago

That's not real ETA, ETA is usually the expected time when the transfer will finish, like 1:31 PM Apr 28th or 13:31 - Apr 28th or something like that...calculated from the "fake" ETA output given by rclone ;-)

Having an option in the preferences to switch to this instead of the current way would be much much more human readable... ;-)

kapitainsky commented 4 years ago

This is how rclone reports ETA - at the moment RB simply displays what rclone outputs. If you would like to change its format you have to modify the line of code I mentioned earlier.