TachibanaGeneralLaboratories / download-navi

Free and Open Source software download manager
GNU General Public License v3.0
839 stars 87 forks source link

[Feature request] Increase UI refresh rate from 2 secs to 0.5 sec #60

Open KaKi87 opened 4 years ago

KaKi87 commented 4 years ago

On the download notification and in the app, the data (progress bar, percentage, speed, etc.) is refreshed every 2 secs.

It is kinda frustrating to not feel real-time refresh, like in ADM for Android or IDM for Windows.

So, I'd like very much to have the UI refreshing every 0.5 sec (500 ms) instead of 2 secs.

Thanks

proninyaroslav commented 4 years ago

I will study this moment. First of all, so that it doesn't cause performance problems.

jwmh commented 4 years ago

Alt suggestion, add as a user-changeable setting. I definitely use an old, slow clunker of Android hardware, so I appreciate things being lightweight.

KaKi87 commented 4 years ago

I can assure you people that refresh rate doesn't affect performance. Just try ADM Holo and you'll find out. Besides, isn't there an Android alternative to JS requestAnimationFrame ?

proninyaroslav commented 4 years ago

@KaKi87 It just requires polling of parts in order to calculate the current progress (if it's a multipart download). I just haven't studied whether this can adversely affect performance.

KaKi87 commented 4 years ago

ADM is also multipart.

proninyaroslav commented 4 years ago

In different apps, different approaches to implementing this :)

KaKi87 commented 3 years ago

Hello, Any news on this ? I would also like to report that multipart download makes speed and ETA indicators behave weirdly, like if each part had its own progress measurement function and would asynchronously display its own values 🤔 Thanks

KrasnayaPloshchad commented 2 years ago

I think set set refresh rate to 1 sec would be better.

KaKi87 commented 2 years ago

1 sec is still a bit low to feel the download progress when you have FTTH, that's why I suggested 0.5 sec. Although the test option would be to refresh the UI instantly when a progress event is fired.

proninyaroslav commented 2 years ago

I'll see if it's possible to change without breaking the ETA and speed calculation.