Skip to content

Conversation

@mengxr
Copy link
Contributor

@mengxr mengxr commented Aug 18, 2015

This PR adds a short description of ml.feature package with code example. The Java package doc will come in a separate PR. @jkbradley

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"most scikit-learn's" --> "most scikit-learn"

@jkbradley
Copy link
Member

Include link to online API doc?

@jkbradley
Copy link
Member

Other than that, LGTM
(I generated it, and ran the example)

@SparkQA
Copy link

SparkQA commented Aug 18, 2015

Test build #41078 has finished for PR 8260 at commit 8b1880b.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@mengxr
Copy link
Contributor Author

mengxr commented Aug 18, 2015

We don't have a dedicated user guide section for ml.feature yet. So I didn't provide a link.

@SparkQA
Copy link

SparkQA commented Aug 18, 2015

Test build #41085 has finished for PR 8260 at commit 425aad2.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@mengxr
Copy link
Contributor Author

mengxr commented Aug 18, 2015

Merged into master and branch-1.5.

@asfgit asfgit closed this in e290029 Aug 18, 2015
asfgit pushed a commit that referenced this pull request Aug 18, 2015
This PR adds a short description of `ml.feature` package with code example. The Java package doc will come in a separate PR. jkbradley

Author: Xiangrui Meng <[email protected]>

Closes #8260 from mengxr/SPARK-7808.

(cherry picked from commit e290029)
Signed-off-by: Xiangrui Meng <[email protected]>
@jkbradley
Copy link
Member

I was suggesting the API guide, not the user guide (but we actually do have a user guide section as well: ml-feature.html)

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.

3 participants