Skip to content
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

Migrates using Image to using Environment in the o16n notebook #964

Conversation

Chris113113
Copy link
Contributor

Description

Creating and deploying a ContainerImage is soon-to-be deprecated functionality inside of AzureML service.

The new recommended flow is instead to prepare an Environment object that contains your Pip/Conda requirements and simply include this in your InferenceConfiguration to be supplied in a Model.deploy() call.

This speeds up image build and service deployment time, and allows subsequent service deployments to leverage existing prepared Environment images.

Related Issues

#962

Checklist:

  • I have followed the contribution guidelines and code style for this project.
  • I have added tests covering my contributions. (Covered by existing tests)
  • [?] I have updated the documentation accordingly. (I have updated in-line documentation in the notebook, not sure if there's other documentation that needs to be updated.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

You'll be able to see Jupyter notebook diff and discuss changes. Powered by ReviewNB.

@miguelgfierro
Copy link
Collaborator

miguelgfierro commented Oct 28, 2019

I get an error when importing the environment dependencies:

from azureml.core import Environment
from azureml.core.environment import CondaDependencies
from azureml.core.model import InferenceConfig
from azureml.core.environment import SparkSection, SparkPackage

Right now we are using azureml-sdk[databricks]==1.0.8 https://github.com/microsoft/recommenders/blob/master/scripts/databricks_install.py#L237
should we change this to a newer version?

@miguelgfierro miguelgfierro changed the base branch from staging to ignite October 28, 2019 19:10
@miguelgfierro
Copy link
Collaborator

@Chris113113 I created a new branch called ignite, rebased to that branch and I will change the version of azureml there

@miguelgfierro miguelgfierro merged commit a90bcdb into recommenders-team:ignite Oct 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants