-
Notifications
You must be signed in to change notification settings - Fork 23
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 Mlflow report example notebook #905
Conversation
@flikka Do you have any notes you think we should add for the AzureML sdk? |
Codecov Report
@@ Coverage Diff @@
## master #905 +/- ##
==========================================
+ Coverage 93.37% 93.46% +0.09%
==========================================
Files 62 62
Lines 2973 2969 -4
==========================================
- Hits 2776 2775 -1
+ Misses 197 194 -3
Continue to review full report at Codecov.
|
d60b618
to
d0c7fd1
Compare
@ryanjdillon closed it? |
8bbd2bb
to
81d5085
Compare
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, a part from the real tags which should be "anonymized" (change the numbers/letters a bit?)
456ab51
to
0483f35
Compare
* Change get_kwargs method to return an empty dict if the secrets environment variable missing
0483f35
to
4a8efa8
Compare
Added the example notebook for querying with the MlFlow client with updates for using the reporters.
closes #861