-
Notifications
You must be signed in to change notification settings - Fork 83
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
Add pipeline properties to API reference #1209
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1209 +/- ##
=======================================
Coverage 99.92% 99.92%
=======================================
Files 196 196
Lines 12007 12013 +6
=======================================
+ Hits 11998 12004 +6
Misses 9 9
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@angela97lin The new template looks awesome! The docstring for feature_importance
is weird though:
I think we should fix in this PR or file an issue to fix it later.
@freddyaboulton Woah, you're right, weird indeed. I've filed #1213 to track updating |
Closes #1198
Updated docs here: https://evalml.alteryx.com/en/1198_classes_attr_docs/api_reference.html#pipelines
Specifically for #1198 here but I figured it would be good to expose some other attributes as well, so I made a new template for pipeline base classes. I also updated
threshold
for BinaryClassificationPipeline to be a property so that we can expose the docstring there.