Skip to content

ML TF-IDF #394

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

Merged
merged 68 commits into from
Mar 25, 2020
Merged

ML TF-IDF #394

merged 68 commits into from
Mar 25, 2020

Conversation

GoEddie
Copy link
Contributor

@GoEddie GoEddie commented Jan 18, 2020

Classes needed to complete the TF-IDF example: https://spark.apache.org/docs/latest/ml-features#tf-idf that completes part of #381

  • Adds Load/Save to Bucketizer
  • HashingTF
  • IDF
  • IDFModel
  • Tokenizer

GOEddieUK and others added 30 commits December 16, 2019 23:25
@GoEddie GoEddie changed the title ML TF-IDF [WIP] ML TF-IDF Mar 2, 2020
@GoEddie GoEddie changed the title [WIP] ML TF-IDF ML TF-IDF Mar 2, 2020
@GoEddie GoEddie changed the title ML TF-IDF [WIP] ML TF-IDF Mar 3, 2020
@GoEddie GoEddie changed the title [WIP] ML TF-IDF ML TF-IDF Mar 3, 2020
@GoEddie
Copy link
Contributor Author

GoEddie commented Mar 17, 2020

@imback82 I don't think there is anything left for me on this to do, if there is let me know (just messaging because I am not sure if you are waiting on me?)

@imback82
Copy link
Contributor

@GoEddie Sorry for the delay. I will get to this this week. Thanks!

Copy link
Contributor

@imback82 imback82 left a comment

Choose a reason for hiding this comment

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

I have few nit comments. But generally LGTM.

Copy link
Contributor

@imback82 imback82 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @GoEddie!

@GoEddie
Copy link
Contributor Author

GoEddie commented Mar 25, 2020

Awesome :)

@imback82 imback82 merged commit a44adc2 into dotnet:master Mar 25, 2020
@MikeRys MikeRys mentioned this pull request May 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants