jeetsukumaran / Syrupy

System Resource Usage Profiler
http://jeetworks.org/programs/syrupy
GNU General Public License v3.0
194 stars 26 forks source link

Use psutil instead of invoking 'ps' #8

Open edisongustavo opened 8 years ago

edisongustavo commented 8 years ago

The psutil.Process (docs) can collect the same information as the ps command.

However, it works on windows too, so Syrupy could be used in that platform too.

What do you think? I will make a pull-request changing the code to use psutil.Process if you think that's nice to have.

RichardBronosky commented 7 years ago

I came here to make this same suggestion. I've been using psutil because a coworker asked me to add a Windows box to the stable and none of my regular DevOps skills did me any good. (Except for my Python development chops.)

jeetsukumaran commented 7 years ago

Sorry for the delay in this. Slipped through the gaps, and just noticed it b/c of @RichardBronosky . Yes, this would be a nice feature, and if you have the code ready, will be happy to accept a PR.