Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[GoldSource] Download secondary progress bar doesn't work properly #2124

Closed
s1lentq opened this issue Mar 29, 2019 · 12 comments
Closed

[GoldSource] Download secondary progress bar doesn't work properly #2124

s1lentq opened this issue Mar 29, 2019 · 12 comments
Assignees

Comments

@s1lentq
Copy link

s1lentq commented Mar 29, 2019

You may notice this when you download a large file using FastDL.
While download a file (e.g >= 100 MB) secondary progress bar always be 100 percent.
DownloadProgressPct_1

Issue is in the function DownloadManager::CheckActiveDownload

DownloadProgressPct_3

DownloadProgressPct_4

In my opinion, a fixes should be like this
DownloadProgressPct_6

Finally, we got the secondary progress bar working correctly
DownloadProgressPct_2

@s1lentq
Copy link
Author

s1lentq commented Mar 29, 2019

#934 related

@fl0werD
Copy link

fl0werD commented Apr 20, 2019

@mikela-valve, please read this topic. Thank you!

@mikela-valve mikela-valve added this to the Next Release milestone Apr 25, 2019
@mikela-valve mikela-valve self-assigned this Apr 25, 2019
@oaus
Copy link

oaus commented Apr 26, 2019

@mikela-valve Is there a possibility to add this feature request #934 (comment) , also if Multithreading is too much work then just should just use one progress bar with file size and download speed with the total amount of files that we will download.

Examples:
download_gif
download_v2
download_v1

@R4to0
Copy link

R4to0 commented May 16, 2019

also if Multithreading is too much work then just should just use one progress bar with file size and download speed with the total amount of files that we will download.

"Multithreading" is a thing since 2013. If you check active TCP connections you'll notice that 3 files are downloaded at same time (it seems it is set to 5 in the engine, not sure). It just needs to be updated to display them all at same time, or like you suggested, show how many files are being downloaded at that moment.

@Matiarguello
Copy link

good gif bb

@mikela-valve
Copy link
Member

A fix to the progress bar not updating properly will be included in the next beta update. I won't be adding additional details to the dialog just yet but will consider it for the future and track work on it in #934 if I do.

@mikela-valve
Copy link
Member

mikela-valve commented May 21, 2019

Fixed in beta 'Exe build: 11:12:36 May 21 2019 (8244)'.

@Mistrick
Copy link

Mistrick commented May 21, 2019

now load bar disappeared
connection to server became very fast but without load bar.
i try connect to server with resourses which i don't have and i load all resourses with black screen, console and servers window
load bar exists with new game local server but disappears with internet servers

@mikela-valve
Copy link
Member

Sorry, just reproduced this based on your info. Part of this request is serviced in Steam and it looks like I was getting a different response based on the version I was using. I'll update beta later with this working.

@mikela-valve
Copy link
Member

This regression should be fixed in the updated beta 'Exe build: 16:32:50 May 22 2019 (8245)'.

@s1lentq
Copy link
Author

s1lentq commented May 23, 2019

I can confirm that this issue is fixed in the current beta

@mikela-valve
Copy link
Member

Closing as fixed.

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

No branches or pull requests

8 participants