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

clear progress bar on exit #1345

Closed
jbenet opened this issue Jun 9, 2015 · 7 comments
Closed

clear progress bar on exit #1345

jbenet opened this issue Jun 9, 2015 · 7 comments
Assignees

Comments

@jbenet
Copy link
Member

jbenet commented Jun 9, 2015

progress bar sometimes lingers after exit:

> ipfs cat QmWmupGwtrGwbu8WL3prFkw4ChAeiYQqHfwRTvBnGhuh6S >/dev/null
75.28 MB / 90.41 MB [=========================================================>-----------] 83.27 % 1s
>

would be ideal to (a) stop the progress bar, (b) clear the line before (c) exiting.

@rht
Copy link
Contributor

rht commented Jun 9, 2015

progress bar sometimes lingers after exit

This could mean either the progress bar keeps adding up even after exit or it hangs there after exit.

(b) clear the line before

The line is to be cleared (with '\b') after exit? But the elapsed time is useful info.
curl/wget retains the progress stat.

@jbenet
Copy link
Member Author

jbenet commented Jun 10, 2015

But the elapsed time is useful info.

Agreed. Maybe we can make it update one last time with all the correct information. that may be the easiest way.

@jbenet
Copy link
Member Author

jbenet commented Jul 23, 2015

this keeps bugging me + confusing people

@jbenet
Copy link
Member Author

jbenet commented Jul 23, 2015

@ipfsbot pin -r QmZjMzrgbiBYs6K97iUsEmiW22kXM62wPnaRtvcSBYMy4Q

I wish o/ worked

@rht
Copy link
Contributor

rht commented Jul 23, 2015

I am confused with the issue with progressbar staying after exit (check curl/wget, they retain the progressbar). There could be extra line saying "data transport/transfer done!" to avoid misunderstanding with the total estimated size.

@rht
Copy link
Contributor

rht commented Jul 24, 2015

i c, like ipfs add's behavior.

@jbenet
Copy link
Member Author

jbenet commented Jul 28, 2015

I am confused with the issue with progressbar staying after exit (check curl/wget, they retain the progressbar). There could be extra line saying "data transport/transfer done!" to avoid misunderstanding with the total estimated size.

yeah if the progress bar stays, it should be drawn to 100% before exiting. the problem now is that (as the picture shows) the bars disappeared midway because the program exited before the bar was redrawn.

i think i'm a bit happier with the bar disappearing, though i'd also be ok with it staying (at 100%). it's not critical output to keep in general, because non-ttys shouldn't see it either. progress bar is mostly for users waiting on the process + "not knowing what it's doing".

thanks for fixing!! 👍

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

No branches or pull requests

2 participants