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

doc/workload: fix double variable in sleep conf #184

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hzhuang1
Copy link

"mean" and "std" property is configured as double type. But if
it's configured as 3000 or 3000.0, it could only be recognized
as int type. Now use 3000.001 to force it as double type.

Signed-off-by: Haojian Zhuang haojian.zhuang@linaro.org

"mean" and "std" property is configured as double type. But if
it's configured as 3000 or 3000.0, it could only be recognized
as int type. Now use 3000.001 to force it as double type.

Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org>
@codecov-io
Copy link

Codecov Report

Merging #184 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #184   +/-   ##
======================================
  Coverage    86.8%   86.8%           
======================================
  Files          33      33           
  Lines         750     750           
  Branches       47      47           
======================================
  Hits          651     651           
  Misses         99      99

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update feef38b...233325e. Read the comment docs.

@ecurtin
Copy link
Contributor

ecurtin commented Nov 28, 2018

Hi @hzhuang1 I see you've opened a bunch of PRs in the last few days. Thanks so much for your contributions!

Spark-Bench is no longer my main focus so I will have to schedule time to review these changes. Hopefully I will be able to get to it this week!

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

Successfully merging this pull request may close these issues.

None yet

3 participants