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: fix invalid workload of data generation lr #181

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

Conversation

hzhuang1
Copy link

This workload will output to /tmp/kmeans-data.csv.

In LinearRegressionDataGen.scala, case class LinearRegressionDataGen()
checks whether the output file is csv file or not. If it's csv file,
stop running the workload immediately. So this workload is impossible
to run, just remove it from document.

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

This workload will output to /tmp/kmeans-data.csv.

In LinearRegressionDataGen.scala, case class LinearRegressionDataGen()
checks whether the output file is csv file or not. If it's csv file,
stop running the workload immediately. So this workload is impossible
to run, just remove it from document.

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

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #181   +/-   ##
======================================
  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 763becb...fc26116. Read the comment docs.

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

2 participants