Hi Jiatong! I have just reviewed your code! Check my comments! BR, Vasileios Papageorgiou https://codereview.appspot.com/330470043/diff/1/HUM/HUM_combine.py ...
6 years, 6 months ago
(2017-10-05 21:31:05 UTC)
#2
https://codereview.appspot.com/330470043/diff/1/HUM/HUM_combine.py File HUM/HUM_combine.py (right): https://codereview.appspot.com/330470043/diff/1/HUM/HUM_combine.py#newcode60 HUM/HUM_combine.py:60: cpu2,mem2=ps_read() It is a bug here.
6 years, 6 months ago
(2017-10-09 18:41:14 UTC)
#4
Hi, I have reviewed your code. https://codereview.appspot.com/330470043/diff/1/HUM/HUM_combine.py File HUM/HUM_combine.py (right): https://codereview.appspot.com/330470043/diff/1/HUM/HUM_combine.py#newcode42 HUM/HUM_combine.py:42: if(number==1): Is number==1 ...
6 years, 6 months ago
(2017-10-09 20:58:33 UTC)
#5
Issue 330470043: new version for HUM including ps/pidstat together, also change code to python function
Created 6 years, 6 months ago by 2016shana2333
Modified 6 years, 6 months ago
Reviewers: michail.xirouchakis_gmail.com, Vasileios Papageorgiou, dkostic, alireza.farshin, younes.tahri3, paul.busi38, engr.sikandar.iqbal
Base URL:
Comments: 6