Skip to content

More tests for TextFeaturizer. #2967

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.

Sign up for GitHub

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

Jump to bottom
Closed
zeahmed opened this issue Mar 14, 2019 · 0 comments · Fixed by #3036
Closed

More tests for TextFeaturizer. #2967

zeahmed opened this issue Mar 14, 2019 · 0 comments · Fixed by #3036
Assignees
@zeahmed
Labels
test related to tests

Comments

@zeahmed
Copy link
Contributor

zeahmed commented Mar 14, 2019

TextFeaturizer is a complex transform and it takes more options as input then any other component in ML.NET. Using those options, the TextFeaturizer build up the underlying components.

Assuming that underlying component are well tested, there is a need to test the options for TextFeaturizer so that we are sure that underlying components are created as intended and results produced are as expected.
@singlis singlis added the test related to tests label Mar 15, 2019
@zeahmed zeahmed mentioned this issue Mar 15, 2019
Merged
@zeahmed zeahmed self-assigned this Mar 19, 2019
This was referenced Mar 19, 2019
Merged
Merged
@ghost ghost locked as resolved and limited conversation to collaborators Mar 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@zeahmed zeahmed

Labels
test related to tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added tests for text featurizer options (Part2). zeahmed/machinelearning
2 participants
@singlis @zeahmed