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

Why was the number "progress bar" removed for fread? #3050

Open
eantonya opened this issue Sep 18, 2018 · 6 comments
Open

Why was the number "progress bar" removed for fread? #3050

eantonya opened this issue Sep 18, 2018 · 6 comments
Labels

Comments

@eantonya
Copy link
Contributor

Just curious why the number progress bar was removed and replaced with the "graphical" progress bar?

Seems to me that this representation shows less info + is fragile to hitting Enter.

@HughParsonage
Copy link
Member

My fault: #2481

#2481 (comment)

@mattdowle
Copy link
Member

mattdowle commented Sep 19, 2018

Maybe the number progress can be brought back. It relies on printing \r. I thought the \r might have been causing the instability as there were historical reports of \r being unstable on RStudio. But it turned out to be unrelated so could try \r again. The current progress bar just prints an extra = each time and has no \r.

@eantonya
Copy link
Contributor Author

@mattdowle I personally don't care about the visual representation, and if people think this is prettier - that's fine with me, but maybe we can have the number printed as well below it or next to it to have more informative output.

@mattdowle
Copy link
Member

mattdowle commented Sep 22, 2018

@eantonya whoa - where did I mention pretty? I did the = to avoid the \r, that was all. Nothing to do with pretty. I started my reply above positively and explained the background to it too. That bug in fread was fraught and lasted weeks related to the multi-threading. It appeared to be Windows-only RStudio-only but turned out to be neither. At the end of the day there is a thread printing that progress inside a parallel region, and calling the R API (printing) is not thread-safe. I think it should be fine to try the \r again but at one point I thought it wouldn't be possible to have any type of text output at all inside the parallel region.

@eantonya
Copy link
Contributor Author

@mattdowle haha, I didn't mean you personally find it pretty - I was simply attempting to cover the case of some people enjoying the current representation. But perhaps I've misjudged, and that group is of size 0.

@mattdowle
Copy link
Member

yep I can't imagine anyone finds it useful. No bounds are even printed so you don't know how many = are going to printed -- you just know it is progressing and that's all. We can try the % again.

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

No branches or pull requests

3 participants