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 FAQ section of docs #997

Merged
merged 3 commits into from
Jul 30, 2020
Merged

Update FAQ section of docs #997

merged 3 commits into from
Jul 30, 2020

Conversation

dsherry
Copy link
Contributor

@dsherry dsherry commented Jul 30, 2020

Changes:

  • Reformat question style
  • Delete empty cell

https://evalml.alteryx.com/en/ds_update_faq/user_guide/faq.html

The headers are too big. We have this problem on the index page too. Its an issue with the pandas style, need to add CSS for it another time.

@dsherry dsherry added the documentation Improvements or additions to documentation label Jul 30, 2020
@codecov
Copy link

codecov bot commented Jul 30, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #997   +/-   ##
=======================================
  Coverage   99.86%   99.86%           
=======================================
  Files         179      179           
  Lines        9424     9424           
=======================================
  Hits         9411     9411           
  Misses         13       13           

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 7b6a469...003f338. Read the comment docs.

@dsherry dsherry marked this pull request as ready for review July 30, 2020 19:41
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 - other than 1 comment about SimpleImputer. Rest of the content looks good.

"\n",
"EvalML optimizes machine learning pipelines on [custom practical objectives](objectives.ipynb) instead of vague machine learning loss functions so that it will find the best pipelines for your specific needs. Furthermore, EvalML [pipelines](pipelines.ipynb) are able to take in all kinds of data (missing values, categorical, etc.) as long as the data are in a single table. EvalML also allows you to build your own pipelines with existing or custom components so you can have more control over the AutoML process. Moreover, EvalML also provides you with support in the form of [data checks](data_checks.ipynb) to ensure that you are aware of potential issues your data may cause with machine learning algorithms.\n",
"\n",
"**How does EvalML handle missing values?**\n",
"#### Q: How does EvalML handle missing values?\n",
"\n",
"EvalML contains imputation components in its pipelines so that missing values are taken care of. EvalML optimizes over different types of imputation to search for the best possible pipeline. You can find more information about components [here](components.ipynb) and in the API reference [here](../generated/evalml.pipelines.components.SimpleImputer.ipynb).\n",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Might need to change this to TypedImputer when the PR merges or just remove the SimpleImputer link.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah good point... will link in the other PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants