-
Notifications
You must be signed in to change notification settings - Fork 21.5k
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
Intel SKLearn Optimization Doc Update for 10/18 webinar #115443
Conversation
@microsoft-github-policy-service agree company="Intel Corporation" |
@raoberman : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Learn Build status updates of commit 36f1051: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
#sign-off |
@sdgilley - still need your approval/sign off before this can be moved forward. Also, NOTE - @raoberman - that this PR adds Jupyter notebook calls and because of that, the commits will need to be moved into the private repo for staging before they can be merged. @balapv @sdgilley - can you do this? let us know if you'd like our assistance. |
The notebook changes have not been merged - Azure/azureml-examples#2429 has some build errors that you'll need to resolve first. Can you coordinate with @balapv to get these fixed and merged? |
@sdgilley @balapv the changes were merged in a different PR, and the tags are in the notebook metadata. not sure why they're not being detected. |
These changes were merged into the private repo. |
@raoberman - I merged your changes in a separate PR. But am reversing the change to the quickstart this morning. The quickstart uses a curated environment, which does not include the required package. The purpose of the quickstart is to get the basics up and running as quickly as possible; it's not the right place to introduce custom environments. So, we can leave your change in the how-to article, but not in the quickstart. |
Adding SKLearnEx sections to existing AzureML documentation so users know how to use SKLearnEx in AzureML before 10/18 Intel+AzureML Webinar
Referenced notebook changes related to PR and already merged: Azure/azureml-examples#2429