Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
44 commits
Select commit Hold shift + click to select a range
87ba4fe
sample commit
Priyanka1329-coder Apr 21, 2025
643c85f
Handled Base MLFlow Image as a Tag in Metadata for Import Model Compo…
Priyanka1329-coder Apr 21, 2025
5e25780
Reverted Changes from preprocess.py
Priyanka1329-coder Apr 21, 2025
0b06057
Code Refactored
Priyanka1329-coder Apr 21, 2025
cd1cf02
Code Refactored
Priyanka1329-coder Apr 21, 2025
47da33e
Code Refactored
Priyanka1329-coder Apr 21, 2025
465fb21
Code Refactored
Priyanka1329-coder Apr 21, 2025
0f68c4d
Test Class fixed
Priyanka1329-coder Apr 22, 2025
b04bfab
Code Refactored
Priyanka1329-coder Apr 22, 2025
911a07a
Code Refactored
Priyanka1329-coder Apr 22, 2025
81afc02
Trimmed Trailing space
Priyanka1329-coder Apr 22, 2025
cf7ae5c
Handled Error AzureML
Priyanka1329-coder Apr 22, 2025
0115149
Code Refactored
Priyanka1329-coder Apr 22, 2025
3b54cdc
Change Reverted
Priyanka1329-coder Apr 22, 2025
f3a8227
Merge branch 'main' into priyanka/mlflow_import_component
Priyanka1329-coder Apr 22, 2025
9ddb34f
Mocked AzureMLOnBehalfOfCredential
Priyanka1329-coder Apr 22, 2025
3d32aef
Merge branch 'priyanka/mlflow_import_component' of https://github.com…
Priyanka1329-coder Apr 22, 2025
351780d
reverted
Priyanka1329-coder Apr 22, 2025
51bb472
reverted
Priyanka1329-coder Apr 22, 2025
d670019
Testing Base MLFlow Image check
Priyanka1329-coder Apr 22, 2025
4c16b82
Latest all changes reverted
Priyanka1329-coder Apr 22, 2025
43fbc90
convertors.py pyfunc changes done
Priyanka1329-coder Apr 22, 2025
208ed56
change done in convertors.py
Priyanka1329-coder Apr 22, 2025
5ac0759
changes done in convertors.py
Priyanka1329-coder Apr 22, 2025
746e714
changes done
Priyanka1329-coder Apr 22, 2025
082aeeb
changes done
Priyanka1329-coder Apr 22, 2025
2944af2
changes fixed
Priyanka1329-coder Apr 22, 2025
3033e41
changes fix
Priyanka1329-coder Apr 22, 2025
9c701fc
testing1
Priyanka1329-coder Apr 22, 2025
fd08542
testing2
Priyanka1329-coder Apr 22, 2025
55894e0
testing3
Priyanka1329-coder Apr 22, 2025
47180ba
testing4
Priyanka1329-coder Apr 22, 2025
b1d2b77
sample commit
Priyanka1329-coder Apr 22, 2025
3b92f62
sample commit1
Priyanka1329-coder Apr 22, 2025
1a6ffd3
sample commit2
Priyanka1329-coder Apr 22, 2025
5260f24
Test1
Priyanka1329-coder Apr 22, 2025
0b3c1fe
test4
Priyanka1329-coder Apr 22, 2025
fa1574f
test5
Priyanka1329-coder Apr 22, 2025
107be1c
Merge branch 'main' of https://github.com/Azure/azureml-assets into p…
Priyanka1329-coder Apr 24, 2025
6d41ff1
Merge branch 'main' into priyanka/mlflow_import_component
Priyanka1329-coder Apr 24, 2025
9c77781
spec.yml changes added
Priyanka1329-coder Apr 24, 2025
3199eeb
Merge branch 'priyanka/mlflow_import_component' of https://github.com…
Priyanka1329-coder Apr 24, 2025
abf4f9e
Chnages done
Priyanka1329-coder Apr 24, 2025
fdd783f
Changes reverted
Priyanka1329-coder Apr 24, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ environment: azureml://registries/azureml/environments/model-management/versions

code: ../../src/
command: |
pip install transformers==4.48.0 torch==2.2.2 numpy==1.23.5 pandas==1.5.3 urllib3==1.26.19 --no-cache-dir --force-reinstall
# TODO: This has one disadvantage as shell logs wont be pushed to appinsights
set -ex
IFS=',' read -ra pip_pkgs <<< "$[[${{inputs.extra_pip_requirements}}]]"
Expand All @@ -39,7 +40,7 @@ inputs:

vllm_enabled:
type: boolean
description: Enable vllm in the converted model
description: Enable vllm in the converted model
default: false
optional: false

Expand Down Expand Up @@ -141,7 +142,7 @@ inputs:
Pip dependencies expressed as below. Do not use quotes for passing.
eg: pkg1==1.0, pkg2, pkg3==1.0
optional: true

inference_base_image:
type: string
description: |
Expand Down
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Priyanka, Why is only the mlflow-model-inference image information included here? Why not the foundation-model-inference as well?

Original file line number Diff line number Diff line change
Expand Up @@ -114,6 +114,14 @@ def _save(
:type metadata: Optional[Dict]. Defaults to {}.
"""
signatures = self._signatures or self.get_model_signature()
# Check and update Base MLFlow Image metadata in Import Model Component
if not self._vllm_enabled:
from azureml.model.mgmt.utils.common_utils import get_mlclient
mlclient = get_mlclient("azureml")
mlFlow_image = mlclient.environments.get("mlflow-model-inference", label="latest")
metadata["azureml.base_image"] = "mcr.microsoft.com/azureml/curated/mlflow-model-inference:" \
+ str(mlFlow_image.version)
logger.info("Metadata: {}".format(metadata))
# set metadata info
metadata.update(fetch_mlflow_acft_metadata(
base_model_name=self._model_id,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -183,6 +183,12 @@ def _save(
metadata["azureml.base_image"] = "mcr.microsoft.com/azureml/curated/foundation-model-inference:" \
+ str(vllm_image.version)
logger.info("Metadata: {}".format(metadata))
else:
mlclient = get_mlclient("azureml")
mlFlow_image = mlclient.environments.get("mlflow-model-inference", label="latest")
metadata["azureml.base_image"] = "mcr.microsoft.com/azureml/curated/mlflow-model-inference:" \
+ str(mlFlow_image.version)
logger.info("Metadata: {}".format(metadata))

if self._model_flavor == "OSS":
try:
Expand Down
Loading