-
Notifications
You must be signed in to change notification settings - Fork 2k
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
PUBDEV-4940 UpliftDRF documentation [nocheck] #5918
Conversation
- added documentation pages - added params/algo to toctree; minor syntax updates, still need to add image to Uplift DRF; rst files only - added image; toctree algo shift - updated api comments - add comma - fix python example, it was failing on assertion error and uplift_model does not have plot_auuc() method - fix test in uplift_metric, it failing on assert - fix all available in
…update main doc page
@hannah-tillman, could you please make a review of this documentation again? I made a few changes of the main page, add more information and change the API a little bit. Thanks a lot! |
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.
just updated -- mostly minor syntax and readability issues! LGTM :)
@maurever handled a few of the suggestions that were doc-centered: shifted the demo to under the intro and merged the references; also updated the one FAQ to say Uplift DRF |
Thanks, @hannah-tillman, you implemented all suggestions! I will just separate back the references and sources, it make sense 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.
Thank you! 🥇 Please remove Uplift DRF from balance_classes and calibration detail hyperparameter pages.
f5e62bd
to
7bf3497
Compare
JIRA: https://h2oai.atlassian.net/browse/PUBDEV-4940
Documentation for UpliftDRF algorithm.
Minor API changes.