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

show proper LFS progress #3171

Closed
shiftkey opened this issue Oct 27, 2017 · 7 comments · Fixed by #9582
Closed

show proper LFS progress #3171

shiftkey opened this issue Oct 27, 2017 · 7 comments · Fixed by #9582
Labels
enhancement help wanted Issues marked as ideal for external contributors

Comments

@shiftkey
Copy link
Member

We made a bunch of progress in #2355 to get the raw LFS progress information in the app. But it's currently just the raw progress, it'd be good to get this information flowing through to the progress side:

  • clone progress bar will fetch LFS assets after
  • checkout progress can include downloading LFS assets
  • fetch/push/pull also
@joshaber joshaber added the design-input-needed Issues that require design input from the core team before the work can be started label Oct 27, 2017
@joshaber
Copy link
Contributor

Note that there's an open design question #2355 (comment):

I've also run into a new and exciting UX problem.

lfs clone downloads files in parallel for ✨ super performance ✨ But this means it's really hard to show the user a unified, reliable progress bar. File X might be 50% done while File Y is 1% done.

We could show progress in terms of how many files are downloaded (that'd be blocked on git-lfs/git-lfs#2599), but that'll be a little misleading, e.g., if we've downloaded all the small files but have 2 huge files left.

@delucis
Copy link

delucis commented Nov 16, 2017

Will this include showing upload progress when publishing a new repo? Currently a slow LFS upload doesn’t seem to trigger anything more than the standard rotating arrows to show that publishing is in progress:

Publishing Repository indicator

@j-f1
Copy link
Contributor

j-f1 commented Nov 16, 2017

What version of Desktop are you running @delucis?

@delucis
Copy link

delucis commented Nov 16, 2017

I just updated to 1.0.9 and I see now although there’s no indicator in the publish modal itself, the progress is shown below “Pushing to origin” in the top bar.

screen shot 2017-11-17 at 0 56 20

My bad, sorry!

@simonseyer
Copy link

simonseyer commented Jun 1, 2018

untitled

When many files are uploaded the file progress is not consistent. As a quick win it may be possible to:

  • Only change the currently processed file number when it is larger than the one before (1/84 -> 30/84 but don't jump back to 1/84)
  • Show the uploaded file size in MB instead of bytes (or don't show it at all since it's switching between the files anyways right now)

@simonseyer
Copy link

I just pushed via the command line and the output was pretty perfect: Git LFS: (4 of 225 files) 75.54 MB / 1.37 GB. Couldn't this output be shown in the UI?

@billygriffin billygriffin removed the design-input-needed Issues that require design input from the core team before the work can be started label Nov 20, 2018
@billygriffin
Copy link
Contributor

Sorry for the delay here. @ampinsk and I discussed this this morning and we both agree that @simonseyer's suggestion to leverage the output displayed on the command line seems totally reasonable. I'm tagging as help wanted to indicate that we'd love for a community contributor to give it a go. Thanks for all the input!

@billygriffin billygriffin added the help wanted Issues marked as ideal for external contributors label Nov 20, 2018
This was referenced Apr 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement help wanted Issues marked as ideal for external contributors
Projects
No open projects
Design review
Awaiting triage
Development

Successfully merging a pull request may close this issue.

7 participants
@joshaber @delucis @shiftkey @simonseyer @billygriffin @j-f1 and others