-
Notifications
You must be signed in to change notification settings - Fork 828
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
feat: add mlflow guidance in website and fix some installation pieces #1428
feat: add mlflow guidance in website and fix some installation pieces #1428
Conversation
FYI: #1407 |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Codecov Report
@@ Coverage Diff @@
## master #1428 +/- ##
==========================================
- Coverage 84.78% 83.13% -1.65%
==========================================
Files 287 280 -7
Lines 14250 14115 -135
Branches 719 719
==========================================
- Hits 12082 11735 -347
- Misses 2168 2380 +212
Continue to review full report at Codecov.
|
website/docs/mlflow/examples.md
Outdated
display(sentiment.transform(df)) | ||
|
||
# Wrap it as a stage in the PipelineModel | ||
sentimentM = PipelineModel(stages=[sentiment]) |
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.
nit: maybe just call this sentiment_model
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.
This is an awesome PR!! only a few nits
website/docs/mlflow/examples.md
Outdated
.setLeafPredictionCol("leafPrediction") | ||
.setFeaturesShapCol("featuresShap")) | ||
|
||
lightbmCM = lightgbmClassifier.fit(df_trans) |
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.
maybe just lightgbm_model
website/docs/mlflow/examples.md
Outdated
|
||
with mlflow.start_run(): | ||
|
||
featureColumns = ["Number of times pregnant","Plasma glucose concentration a 2 hours in an oral glucose tolerance test", |
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.
nit: snake case in python examples
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Summary
Add samples of Mlflow usage with SynapseML on the website and fix some installation guidance for the
next
version.Tests
Tested website rendering correctly.