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

Questions on SO: pipe #287

Closed
lrq3000 opened this issue Oct 11, 2016 · 4 comments
Closed

Questions on SO: pipe #287

lrq3000 opened this issue Oct 11, 2016 · 4 comments

Comments

@lrq3000
Copy link
Member

lrq3000 commented Oct 11, 2016

The best alternative to tqdm pipe: https://github.com/Xfennec/progress

From its readme:

How does it work?

It simply scans /proc for interesting commands, and then looks at directories fd and fdinfo to find opened files and seek positions, and reports status for the largest file.

It's very light, and compatible with virtually any command.

But it depends on ncurses and works only on Linux.

@casperdcl
Copy link
Sponsor Member

Hello world. I'm sort of back (i.e. might have time to look at tqdm again). I've answered http://unix.stackexchange.com/questions/1537/measure-pipe-throughput-in-the-shell/316217
... we could try re-implementing the process monitoring things that UNIX has, but it won't be windows compatible.

@lrq3000
Copy link
Member Author

lrq3000 commented Oct 13, 2016

Hey Casper, good to see you back :)

I see no issue in implementing a platform specific additional feature as long as it doesn't prevent the main feature, tqdm pipe, from being used on Windows. Also, if you use psutils for process monitoring, it might also work on Windows if psutils for windows are installed, so we might potentially find a way later on to port this kind of platform-specific code.

Also you come back at the right time: I was going to work on merging tqdm PRs this week-end, starting tomorrow, because I won't be able to get back to tqdm for a few weeks at least after this week-end. If you can check the PRs (tagged to-review), that would be great!

My plan was to first merge the bugfix PRs, or PRs adding small features to the core. Then release a new minor version of tqdm with these fixes. Then merge the new modular architecture, and then merge submodules PRs (after being rebased on the new modular architecture). Then we can release a new major version of tqdm. What do you think about it? I think it's the cleanest way to move to the new architecture, even if I will lose some time rebasing every submodules.

@lrq3000 lrq3000 closed this as completed Oct 16, 2016
@lrq3000
Copy link
Member Author

lrq3000 commented Oct 16, 2016

@casperdcl I saw you enabled reviews on this repo (or was it enabled automatically?), so I can't merge in my own PRs anymore (which is a good thing, I don't want to disable the reviews feature).

If you want to have a quick look at the things you have to review, you can use this. When you will have reviewed these PRs (except the monitoring thread, it's not urgent and can be done later), I will implement the new submodules architecture #198 and then we can offer Dask support #279 and other submodules in the standard pypi release in a safe manner.

@casperdcl
Copy link
Sponsor Member

Nice, thanks Stephen.

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