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

Update evalml docs to mention woodwork #1361

Merged
merged 5 commits into from
Oct 29, 2020
Merged

Update evalml docs to mention woodwork #1361

merged 5 commits into from
Oct 29, 2020

Conversation

dsherry
Copy link
Contributor

@dsherry dsherry commented Oct 28, 2020

Fix #1287

Adds a mention and example to the start page and to the automl user guide page about woodwork.

This is just to start us off. Eventually I'd like us to add a page or section to the automl guide about "formatting data for automl", which shows how to set the column types using woodwork, override some of the default inference and run automl with those settings.

@codecov
Copy link

codecov bot commented Oct 28, 2020

Codecov Report

Merging #1361 into main will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1361   +/-   ##
=======================================
  Coverage   99.95%   99.95%           
=======================================
  Files         213      213           
  Lines       13857    13857           
=======================================
  Hits        13850    13850           
  Misses          7        7           

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 4166fbb...6d87a54. Read the comment docs.

@dsherry
Copy link
Contributor Author

dsherry commented Oct 29, 2020

Docs visible here for the start page

"execution_count": null,
"metadata": {},
"outputs": [],
"source": []
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops, will delete this

Copy link
Collaborator

@jeremyliweishih jeremyliweishih left a comment

Choose a reason for hiding this comment

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

lgtm!

docs/source/start.ipynb Outdated Show resolved Hide resolved
docs/source/user_guide/automl.ipynb Outdated Show resolved Hide resolved
"source": [
"To provide data to EvalML, it is recommended that you create a `DataTable` object using [the woodwork project](https://woodwork.alteryx.com/en/stable/).\n",
"\n",
"EvalML also accepts and works well with pandas `DataFrames`. But using the `DataTable` makes it easy to control how EvalML will treat each feature, as a numeric feature, a categorical feature, a text feature or other type of feature. Woodwork `DataTable`s include features like inferring when a categorical feature should be treated as a text feature. For this reason, if you don't provide woodwork objects, EvalML will raise a warning."
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"EvalML also accepts and works well with pandas `DataFrames`. But using the `DataTable` makes it easy to control how EvalML will treat each feature, as a numeric feature, a categorical feature, a text feature or other type of feature. Woodwork `DataTable`s include features like inferring when a categorical feature should be treated as a text feature. For this reason, if you don't provide woodwork objects, EvalML will raise a warning."
"EvalML also accepts and works well with pandas `DataFrames`. But using the `DataTable` makes it easy to control how EvalML will treat each feature, as a numeric feature, a categorical feature, a text feature or other type of feature. Woodwork's DataTable includes features like inferring when a categorical feature should be treated as a natural language feature. For this reason, if you don't provide woodwork objects, EvalML will raise a warning."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@gsheni ah right. I think we're gonna stick with "text" in evalml for the time being. Some of our classes use "text". We can have a separate conversation about that if you'd like.

I'll add in your other suggestion.

Thanks!

docs/source/user_guide/automl.ipynb Outdated Show resolved Hide resolved
gsheni
gsheni previously approved these changes Oct 29, 2020
@gsheni
Copy link
Contributor

gsheni commented Oct 29, 2020

@dsherry Formating
Screen Shot 2020-10-29 at 1 08 21 PM

@gsheni gsheni self-requested a review October 29, 2020 17:09
@gsheni gsheni dismissed their stale review October 29, 2020 17:10

re-reviewed and found some things.

@dsherry dsherry merged commit 77ae097 into main Oct 29, 2020
@dsherry dsherry deleted the ds_1287_woodwork_docs branch October 29, 2020 17:32
@dsherry dsherry mentioned this pull request Oct 29, 2020
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.

Add documentation for Woodwork support
3 participants