Combined variance - Githubissues

benchflow / analysers

Spark scripts utilised to analyse data and compute performance metrics

Other

0 stars 1 forks source link

Combined variance #85

Closed Cerfoglg closed 8 years ago

Cerfoglg commented 8 years ago

Fixes #83

[x] File: analysers/commons/commons.py:L17-36
<img border=0 src='https://avatars.githubusercontent.com/u/7163641?v=3' height=16 width=16'> Update the method with the filename instead of the SUTName
[x] File: analysers/experiments/cpu.py:L68-85
<img border=0 src='https://avatars.githubusercontent.com/u/7163641?v=3' height=16 width=16'> Should be in commons. Try to abstract the function enough so that it can be placed in commons.
[x] File: analysers/commons/commons.py:L236-261
<img border=0 src='https://avatars.githubusercontent.com/u/7163641?v=3' height=16 width=16'> This can be moved out of the if statement
[x] File: analysers/commons/commons.py:L236-261
<img border=0 src='https://avatars.githubusercontent.com/u/7163641?v=3' height=16 width=16'> This can be moved out of the if statement
[x] File: analysers/commons/commons.py:L236-261
<img border=0 src='https://avatars.githubusercontent.com/u/7163641?v=3' height=16 width=16'> This can be moved out of the if statement
[x] File: analysers/commons/commons.py:L236-261
<img border=0 src='https://avatars.githubusercontent.com/u/7163641?v=3' height=16 width=16'> This can be moved out of the if statement
[x] File: analysers/commons/commons.py:L236-261
<img border=0 src='https://avatars.githubusercontent.com/u/7163641?v=3' height=16 width=16'> This can be moved out of the if statement
[x] File: analysers/commons/commons.py:L236-261
<img border=0 src='https://avatars.githubusercontent.com/u/7163641?v=3' height=16 width=16'> This can be moved out of the if statement
[x] General Comment
<img border=0 src='https://avatars.githubusercontent.com/u/8819678?v=3' height=16 width=16'> @VincenzoFerme All good to go

Cerfoglg commented 8 years ago

@VincenzoFerme All good to go

VincenzoFerme commented 8 years ago

:+1: