FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat(automl): add support for feature importance by helinwang · Pull Request #9816 · googleapis/google-cloud-python · GitHub

feat(automl): add support for feature importance - #9816

Merged
busunkim96 merged 1 commit into
googleapis:masterfrom
helinwang:feature_importance
Nov 16, 2019
Merged

feat(automl): add support for feature importance#9816
busunkim96 merged 1 commit into
googleapis:masterfrom
helinwang:feature_importance

Conversation

Copy link
Copy Markdown
Contributor

Previously feature importance is added with a params parameter, the
user has to set params = {"feature_importance": "true"}. This PR
simplifies the logic, the user just have to pass feature_importance =
True.

googlebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 15, 2019
helinwang requested a review from sirtorry November 15, 2019 23:41
Previously feature importance is added with a params parameter, the
user has to set params = {"feature_importance": "true"}. This PR
simplifies the logic, the user just have to pass feature_importance =
True.

sirtorry left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

lgtm

busunkim96 merged commit 8d745d0 into googleapis:master Nov 16, 2019
parthea pushed a commit that referenced this pull request Oct 21, 2023
Previously feature importance is added with a params parameter, the
user has to set params = {"feature_importance": "true"}. This PR
simplifies the logic, the user just have to pass feature_importance =
True.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL