arunoda / node-usage

process usage lookup with nodejs
MIT License
389 stars 93 forks source link

memory is incorrect for osx #9

Closed leonchen closed 10 years ago

leonchen commented 11 years ago

the rss from ps in osx is "in 1024 byte units": https://developer.apple.com/library/mac/#documentation/Darwin/Reference/Manpages/man1/ps.1.html

arunoda commented 11 years ago

@jhurliman Can you please look into this?

jhurliman commented 11 years ago

https://github.com/arunoda/node-usage/pull/10

arunoda commented 11 years ago

fixed and published in version 0.3.8

jhurliman commented 11 years ago

Can this be closed?

arunoda commented 11 years ago

I hope we could, just waited for the @leonchen 's confirmation. Anyway it works.

arunoda commented 10 years ago

I can confirmed this is fixed. And I refactored the ps provider.