-
Notifications
You must be signed in to change notification settings - Fork 91
Add partial dependence to API reference #1537
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
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1537 +/- ##
=======================================
Coverage 100.0% 100.0%
=======================================
Files 232 232
Lines 16639 16639
=======================================
Hits 16631 16631
Misses 8 8 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.
nice change!
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! Should we make _get_prediction_vs_actual_data
public? What do you think @bchen1116 ?
@angela97lin @freddyaboulton We can make |
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 great! Thanks for cleaning this up
@freddyaboulton @bchen1116 Looks like unit tests are missing for |
Closes #1516
Also does minor reorganization, separating out data utils from graphing utils, but open to other suggestions as to how to split up that section more 🤷