truefoundry 0.5.3rc1__tar.gz → 0.5.3rc2__tar.gz

{truefoundry-0.5.3rc1 → truefoundry-0.5.3rc2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: truefoundry
-Version: 0.5.3rc1
+Version: 0.5.3rc2
 Summary: Truefoundry CLI
 Author: Abhishek Choudhary
 Author-email: abhishek@truefoundry.com

{truefoundry-0.5.3rc1 → truefoundry-0.5.3rc2}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "truefoundry"
-version = "0.5.3rc1"
+version = "0.5.3rc2"
 description = "Truefoundry CLI"
 authors = ["Abhishek Choudhary <abhishek@truefoundry.com>"]
 readme = "README.md"

{truefoundry-0.5.3rc1 → truefoundry-0.5.3rc2}/truefoundry/ml/autogen/client/__init__.py

@@ -184,6 +184,9 @@ from truefoundry.ml.autogen.client.models.finalize_artifact_version_request_dto
     FinalizeArtifactVersionRequestDto,
 )
 from truefoundry.ml.autogen.client.models.framework import Framework
+from truefoundry.ml.autogen.client.models.get_artifact_tags_response_dto import (
+    GetArtifactTagsResponseDto,
+)
 from truefoundry.ml.autogen.client.models.get_experiment_response_dto import (
     GetExperimentResponseDto,
 )

{truefoundry-0.5.3rc1 → truefoundry-0.5.3rc2}/truefoundry/ml/autogen/client/api/mlfoundry_artifacts_api.py

@@ -79,6 +79,9 @@ from truefoundry.ml.autogen.client.models.export_deployment_files_request_dto im
 from truefoundry.ml.autogen.client.models.finalize_artifact_version_request_dto import (
     FinalizeArtifactVersionRequestDto,
 )
+from truefoundry.ml.autogen.client.models.get_artifact_tags_response_dto import (
+    GetArtifactTagsResponseDto,
+)
 from truefoundry.ml.autogen.client.models.get_signed_url_for_dataset_write_request_dto import (
     GetSignedURLForDatasetWriteRequestDto,
 )
@@ -5338,6 +5341,150 @@ class MlfoundryArtifactsApi:
             _request_auth=_params.get("_request_auth"),
         )
 
+    @validate_arguments
+    def get_tags_for_artifact_get(
+        self, artifact_id: StrictStr, **kwargs
+    ) -> GetArtifactTagsResponseDto:  # noqa: E501
+        """Get Tags For Artifact  # noqa: E501
+
+        This method makes a synchronous HTTP request by default. To make an
+        asynchronous HTTP request, please pass async_req=True
+
+        >>> thread = api.get_tags_for_artifact_get(artifact_id, async_req=True)
+        >>> result = thread.get()
+
+        :param artifact_id: (required)
+        :type artifact_id: str
+        :param async_req: Whether to execute the request asynchronously.
+        :type async_req: bool, optional
+        :param _request_timeout: timeout setting for this request.
+               If one number provided, it will be total request
+               timeout. It can also be a pair (tuple) of
+               (connection, read) timeouts.
+        :return: Returns the result object.
+                 If the method is called asynchronously,
+                 returns the request thread.
+        :rtype: GetArtifactTagsResponseDto
+        """
+        kwargs["_return_http_data_only"] = True
+        if "_preload_content" in kwargs:
+            message = "Error! Please call the get_tags_for_artifact_get_with_http_info method with `_preload_content` instead and obtain raw data from ApiResponse.raw_data"  # noqa: E501
+            raise ValueError(message)
+        return self.get_tags_for_artifact_get_with_http_info(artifact_id, **kwargs)  # noqa: E501
+
+    @validate_arguments
+    def get_tags_for_artifact_get_with_http_info(
+        self, artifact_id: StrictStr, **kwargs
+    ) -> ApiResponse:  # noqa: E501
+        """Get Tags For Artifact  # noqa: E501
+
+        This method makes a synchronous HTTP request by default. To make an
+        asynchronous HTTP request, please pass async_req=True
+
+        >>> thread = api.get_tags_for_artifact_get_with_http_info(artifact_id, async_req=True)
+        >>> result = thread.get()
+
+        :param artifact_id: (required)
+        :type artifact_id: str
+        :param async_req: Whether to execute the request asynchronously.
+        :type async_req: bool, optional
+        :param _preload_content: if False, the ApiResponse.data will
+                                 be set to none and raw_data will store the
+                                 HTTP response body without reading/decoding.
+                                 Default is True.
+        :type _preload_content: bool, optional
+        :param _return_http_data_only: response data instead of ApiResponse
+                                       object with status code, headers, etc
+        :type _return_http_data_only: bool, optional
+        :param _request_timeout: timeout setting for this request. If one
+                                 number provided, it will be total request
+                                 timeout. It can also be a pair (tuple) of
+                                 (connection, read) timeouts.
+        :param _request_auth: set to override the auth_settings for an a single
+                              request; this effectively ignores the authentication
+                              in the spec for a single request.
+        :type _request_auth: dict, optional
+        :type _content_type: string, optional: force content-type for the request
+        :return: Returns the result object.
+                 If the method is called asynchronously,
+                 returns the request thread.
+        :rtype: tuple(GetArtifactTagsResponseDto, status_code(int), headers(HTTPHeaderDict))
+        """
+
+        _params = locals()
+
+        _all_params = ["artifact_id"]
+        _all_params.extend(
+            [
+                "async_req",
+                "_return_http_data_only",
+                "_preload_content",
+                "_request_timeout",
+                "_request_auth",
+                "_content_type",
+                "_headers",
+            ]
+        )
+
+        # validate the arguments
+        for _key, _val in _params["kwargs"].items():
+            if _key not in _all_params:
+                raise ApiTypeError(
+                    "Got an unexpected keyword argument '%s'"
+                    " to method get_tags_for_artifact_get" % _key
+                )
+            _params[_key] = _val
+        del _params["kwargs"]
+
+        _collection_formats = {}
+
+        # process the path parameters
+        _path_params = {}
+
+        # process the query parameters
+        _query_params = []
+        if _params.get("artifact_id") is not None:  # noqa: E501
+            _query_params.append(("artifact_id", _params["artifact_id"]))
+
+        # process the header parameters
+        _header_params = dict(_params.get("_headers", {}))
+        # process the form parameters
+        _form_params = []
+        _files = {}
+        # process the body parameter
+        _body_params = None
+        # set the HTTP header `Accept`
+        _header_params["Accept"] = self.api_client.select_header_accept(
+            ["application/json"]
+        )  # noqa: E501
+
+        # authentication setting
+        _auth_settings = ["HTTPBearer", "APIKeyCookie"]  # noqa: E501
+
+        _response_types_map = {
+            "200": "GetArtifactTagsResponseDto",
+            "422": "HTTPValidationError",
+        }
+
+        return self.api_client.call_api(
+            "/api/2.0/mlflow/mlfoundry-artifacts/artifacts/get-tags",
+            "GET",
+            _path_params,
+            _query_params,
+            _header_params,
+            body=_body_params,
+            post_params=_form_params,
+            files=_files,
+            response_types_map=_response_types_map,
+            auth_settings=_auth_settings,
+            async_req=_params.get("async_req"),
+            _return_http_data_only=_params.get("_return_http_data_only"),  # noqa: E501
+            _preload_content=_params.get("_preload_content", True),
+            _request_timeout=_params.get("_request_timeout"),
+            collection_formats=_collection_formats,
+            _request_auth=_params.get("_request_auth"),
+        )
+
     @validate_arguments
     def list_artifact_versions_post(
         self,

{truefoundry-0.5.3rc1 → truefoundry-0.5.3rc2}/truefoundry/ml/autogen/client/models/__init__.py

@@ -155,6 +155,9 @@ from truefoundry.ml.autogen.client.models.finalize_artifact_version_request_dto
     FinalizeArtifactVersionRequestDto,
 )
 from truefoundry.ml.autogen.client.models.framework import Framework
+from truefoundry.ml.autogen.client.models.get_artifact_tags_response_dto import (
+    GetArtifactTagsResponseDto,
+)
 from truefoundry.ml.autogen.client.models.get_experiment_response_dto import (
     GetExperimentResponseDto,
 )

{truefoundry-0.5.3rc1 → truefoundry-0.5.3rc2}/truefoundry/ml/autogen/client/models/artifact_version_manifest.py

@@ -24,6 +24,7 @@ from truefoundry.pydantic_v1 import (
     Field,
     StrictStr,
     conint,
+    conlist,
     constr,
     validator,
 )
@@ -42,12 +43,16 @@ class ArtifactVersionManifest(BaseModel):
         default=...,
         description="+label=Metadata +docs=Metadata for the artifact or model version +usage=Metadata for the artifact or model version +uiType=JsonInput",
     )
+    tags: Optional[conlist(constr(strict=True, max_length=128))] = Field(
+        default=None,
+        description="+label=Tags +usage=Tags for the artifact or model version +docs=Tags for the artifact or model version",
+    )
     type: Optional[StrictStr] = "artifact-version"
     source: Source = Field(...)
     step: Optional[conint(strict=True, ge=0)] = Field(
         default=0, description="+label=Step"
     )
-    __properties = ["description", "metadata", "type", "source", "step"]
+    __properties = ["description", "metadata", "tags", "type", "source", "step"]
 
     @validator("type")
     def type_validate_enum(cls, value):
@@ -99,6 +104,7 @@ class ArtifactVersionManifest(BaseModel):
             {
                 "description": obj.get("description"),
                 "metadata": obj.get("metadata"),
+                "tags": obj.get("tags"),
                 "type": obj.get("type")
                 if obj.get("type") is not None
                 else "artifact-version",

truefoundry-0.5.3rc2/truefoundry/ml/autogen/client/models/get_artifact_tags_response_dto.py

@@ -0,0 +1,65 @@
+# coding: utf-8
+
+"""
+FastAPI
+
+No description provided (generated by Openapi Generator https://github.com/openapitools/openapi-generator)
+
+The version of the OpenAPI document: 0.1.0
+Generated by OpenAPI Generator (https://openapi-generator.tech)
+
+Do not edit the class manually.
+"""  # noqa: E501
+
+from __future__ import annotations
+
+import json
+import pprint
+import re  # noqa: F401
+
+from truefoundry.pydantic_v1 import BaseModel, Field, StrictStr, conlist
+
+
+class GetArtifactTagsResponseDto(BaseModel):
+    """
+    GetArtifactTagsResponseDto
+    """
+
+    tags: conlist(StrictStr) = Field(...)
+    __properties = ["tags"]
+
+    class Config:
+        """Pydantic configuration"""
+
+        allow_population_by_field_name = True
+        validate_assignment = True
+
+    def to_str(self) -> str:
+        """Returns the string representation of the model using alias"""
+        return pprint.pformat(self.dict(by_alias=True))
+
+    def to_json(self) -> str:
+        """Returns the JSON representation of the model using alias"""
+        return json.dumps(self.to_dict())
+
+    @classmethod
+    def from_json(cls, json_str: str) -> GetArtifactTagsResponseDto:
+        """Create an instance of GetArtifactTagsResponseDto from a JSON string"""
+        return cls.from_dict(json.loads(json_str))
+
+    def to_dict(self):
+        """Returns the dictionary representation of the model using alias"""
+        _dict = self.dict(by_alias=True, exclude={}, exclude_none=True)
+        return _dict
+
+    @classmethod
+    def from_dict(cls, obj: dict) -> GetArtifactTagsResponseDto:
+        """Create an instance of GetArtifactTagsResponseDto from a dict"""
+        if obj is None:
+            return None
+
+        if not isinstance(obj, dict):
+            return GetArtifactTagsResponseDto.parse_obj(obj)
+
+        _obj = GetArtifactTagsResponseDto.parse_obj({"tags": obj.get("tags")})
+        return _obj

{truefoundry-0.5.3rc1 → truefoundry-0.5.3rc2}/truefoundry/ml/autogen/client/models/model_version_manifest.py

@@ -28,6 +28,7 @@ from truefoundry.pydantic_v1 import (
     Field,
     StrictStr,
     conint,
+    conlist,
     constr,
     validator,
 )
@@ -46,6 +47,10 @@ class ModelVersionManifest(BaseModel):
         default=...,
         description="+label=Metadata +docs=Metadata for the artifact or model version +usage=Metadata for the artifact or model version +uiType=JsonInput",
     )
+    tags: Optional[conlist(constr(strict=True, max_length=128))] = Field(
+        default=None,
+        description="+label=Tags +usage=Tags for the artifact or model version +docs=Tags for the artifact or model version",
+    )
     type: Optional[StrictStr] = "model-version"
     source: Source1 = Field(...)
     framework: Optional[Framework] = None
@@ -56,6 +61,7 @@ class ModelVersionManifest(BaseModel):
     __properties = [
         "description",
         "metadata",
+        "tags",
         "type",
         "source",
         "framework",
@@ -119,6 +125,7 @@ class ModelVersionManifest(BaseModel):
             {
                 "description": obj.get("description"),
                 "metadata": obj.get("metadata"),
+                "tags": obj.get("tags"),
                 "type": obj.get("type")
                 if obj.get("type") is not None
                 else "model-version",

{truefoundry-0.5.3rc1 → truefoundry-0.5.3rc2}/truefoundry/ml/autogen/client_README.md

@@ -121,6 +121,7 @@ Class | Method | HTTP request | Description
 *MlfoundryArtifactsApi* | [**get_signed_urls_for_dataset_write_post**](truefoundry/ml/autogen/client/docs/MlfoundryArtifactsApi.md#get_signed_urls_for_dataset_write_post) | **POST** /api/2.0/mlflow/mlfoundry-artifacts/datasets/get-signed-urls-for-write | Get Signed Urls For Dataset Write
 *MlfoundryArtifactsApi* | [**get_signed_urls_for_read_post**](truefoundry/ml/autogen/client/docs/MlfoundryArtifactsApi.md#get_signed_urls_for_read_post) | **POST** /api/2.0/mlflow/mlfoundry-artifacts/artifact-versions/get-signed-urls-for-read | Get Signed Urls For Read
 *MlfoundryArtifactsApi* | [**get_signed_urls_for_write_post**](truefoundry/ml/autogen/client/docs/MlfoundryArtifactsApi.md#get_signed_urls_for_write_post) | **POST** /api/2.0/mlflow/mlfoundry-artifacts/artifact-versions/get-signed-urls-for-write | Get Signed Urls For Write
+*MlfoundryArtifactsApi* | [**get_tags_for_artifact_get**](truefoundry/ml/autogen/client/docs/MlfoundryArtifactsApi.md#get_tags_for_artifact_get) | **GET** /api/2.0/mlflow/mlfoundry-artifacts/artifacts/get-tags | Get Tags For Artifact
 *MlfoundryArtifactsApi* | [**list_artifact_versions_post**](truefoundry/ml/autogen/client/docs/MlfoundryArtifactsApi.md#list_artifact_versions_post) | **POST** /api/2.0/mlflow/mlfoundry-artifacts/artifact-versions/list | List Artifact Versions
 *MlfoundryArtifactsApi* | [**list_artifacts_post**](truefoundry/ml/autogen/client/docs/MlfoundryArtifactsApi.md#list_artifacts_post) | **POST** /api/2.0/mlflow/mlfoundry-artifacts/artifacts/list | List Artifacts
 *MlfoundryArtifactsApi* | [**list_datasets_post**](truefoundry/ml/autogen/client/docs/MlfoundryArtifactsApi.md#list_datasets_post) | **POST** /api/2.0/mlflow/mlfoundry-artifacts/datasets/list | List Datasets
@@ -219,6 +220,7 @@ Class | Method | HTTP request | Description
  - [FileInfoDto](truefoundry/ml/autogen/client/docs/FileInfoDto.md)
  - [FinalizeArtifactVersionRequestDto](truefoundry/ml/autogen/client/docs/FinalizeArtifactVersionRequestDto.md)
  - [Framework](truefoundry/ml/autogen/client/docs/Framework.md)
+ - [GetArtifactTagsResponseDto](truefoundry/ml/autogen/client/docs/GetArtifactTagsResponseDto.md)
  - [GetExperimentResponseDto](truefoundry/ml/autogen/client/docs/GetExperimentResponseDto.md)
  - [GetLatestRunLogResponseDto](truefoundry/ml/autogen/client/docs/GetLatestRunLogResponseDto.md)
  - [GetMetricHistoryResponse](truefoundry/ml/autogen/client/docs/GetMetricHistoryResponse.md)

{truefoundry-0.5.3rc1 → truefoundry-0.5.3rc2}/truefoundry/ml/autogen/entities/artifacts.py

@@ -1,6 +1,6 @@
 # generated by datamodel-codegen:
 #   filename:  artifacts.json
-#   timestamp: 2024-12-09T09:04:12+00:00
+#   timestamp: 2024-12-16T06:57:32+00:00
 
 from __future__ import annotations
 
@@ -58,6 +58,17 @@ class BaseArtifactVersion(BaseModel):
         ...,
         description="+label=Metadata\n+docs=Metadata for the artifact or model version\n+usage=Metadata for the artifact or model version\n+uiType=JsonInput",
     )
+    tags: Optional[
+        List[
+            constr(
+                regex=r"^(?!^\d+$)(?!.*[._-]{2,})[a-zA-Z0-9]+([._-][a-zA-Z0-9]+)*$",
+                max_length=128,
+            )
+        ]
+    ] = Field(
+        None,
+        description="+label=Tags\n+usage=Tags for the artifact or model version\n+docs=Tags for the artifact or model version",
+    )
 
 
 class MimeType(str, Enum):

{truefoundry-0.5.3rc1 → truefoundry-0.5.3rc2}/truefoundry/ml/log_types/artifacts/artifact.py

@@ -5,11 +5,10 @@ import os
 import tempfile
 import typing
 import uuid
+import warnings
 from pathlib import Path
 from typing import TYPE_CHECKING, Any, Dict, List, NamedTuple, Optional, Union
 
-from pydantic import StrictStr
-
 from truefoundry.ml.artifact.truefoundry_artifact_repo import (
     ArtifactIdentifier,
     MlFoundryArtifactsRepository,
@@ -43,7 +42,7 @@ from truefoundry.ml.log_types.artifacts.utils import (
 )
 from truefoundry.ml.logger import logger
 from truefoundry.ml.session import _get_api_client
-from truefoundry.pydantic_v1 import BaseModel, Extra
+from truefoundry.pydantic_v1 import BaseModel, Extra, StrictStr
 
 if TYPE_CHECKING:
     from truefoundry.ml.mlfoundry_run import MlFoundryRun
@@ -84,6 +83,7 @@ class ArtifactVersion:
         self._deleted = False
         self._description: str = ""
         self._metadata: Dict[str, Any] = {}
+        self._tags: List[str] = []
         self._set_mutable_attrs()
 
     @classmethod
@@ -131,9 +131,13 @@ class ArtifactVersion:
             manifest = self._artifact_version.manifest.actual_instance
             self._description = manifest.description or ""
             self._metadata = copy.deepcopy(manifest.metadata)
+            self._tags = copy.deepcopy(manifest.tags or [])
         else:
             self._description = self._artifact_version.description or ""
-            self._metadata = copy.deepcopy(self._artifact_version.artifact_metadata)
+            self._metadata = copy.deepcopy(
+                self._artifact_version.artifact_metadata or {}
+            )
+            self._tags = []
 
     def _refetch_artifact_version(self, reset_mutable_attrs: bool = True):
         _artifact_version = (
@@ -205,6 +209,33 @@ class ArtifactVersion:
         _validate_artifact_metadata(value)
         self._metadata = copy.deepcopy(value)
 
+    @property
+    def tags(self) -> List[str]:
+        """
+        Get tags for the current model version
+        """
+        if not self._artifact_version.manifest:
+            warnings.warn(
+                message="This model version was created using an older serialization format. tags do not exist, returning empty list",
+                category=DeprecationWarning,
+                stacklevel=2,
+            )
+        return self._tags
+
+    @tags.setter
+    def tags(self, value: List[str]):
+        """
+        Set the tags for current model version
+        """
+        if not self._artifact_version.manifest:
+            warnings.warn(
+                message="This model version was created using an older serialization format. Tags will not be updated",
+                category=DeprecationWarning,
+                stacklevel=2,
+            )
+            return
+        self._tags = copy.deepcopy(value)
+
     @property
     def created_at(self) -> Optional[datetime.datetime]:
         """Get the time at which artifact was created"""
@@ -367,6 +398,7 @@ class ArtifactVersion:
             assert isinstance(manifest, ArtifactVersionManifest)
             manifest.description = self.description
             manifest.metadata = self.metadata
+            manifest.tags = self.tags
         else:
             manifest = None
         try:

{truefoundry-0.5.3rc1 → truefoundry-0.5.3rc2}/truefoundry/ml/log_types/artifacts/model.py

@@ -8,7 +8,7 @@ import typing
 import uuid
 import warnings
 from pathlib import Path
-from typing import TYPE_CHECKING, Any, Dict, Optional, Union
+from typing import TYPE_CHECKING, Any, Dict, List, Optional, Union
 
 from truefoundry.ml.artifact.truefoundry_artifact_repo import (
     ArtifactIdentifier,
@@ -52,7 +52,7 @@ from truefoundry.ml.model_framework import (
     auto_update_model_framework_details,
 )
 from truefoundry.ml.session import _get_api_client
-from truefoundry.pydantic_v1 import BaseModel, Extra
+from truefoundry.pydantic_v1 import BaseModel, Extra, parse_obj_as
 
 if TYPE_CHECKING:
     import numpy as np
@@ -108,6 +108,7 @@ class ModelVersion:
         self._metadata: Dict[str, Any] = {}
         self._environment: Optional[ModelVersionEnvironment] = None
         self._framework: Optional[ModelFrameworkType] = None
+        self._tags: List[str] = []
         self._set_mutable_attrs()
 
     @classmethod
@@ -147,21 +148,26 @@ class ModelVersion:
 
     def _set_mutable_attrs(self):
         if self._model_version.manifest:
-            self._description = self._model_version.manifest.description or ""
-            self._metadata = copy.deepcopy(self._model_version.manifest.metadata)
-            self._environment = copy.deepcopy(self._model_version.manifest.environment)
+            manifest = self._model_version.manifest
+            self._description = manifest.description or ""
+            self._metadata = copy.deepcopy(manifest.metadata)
+            self._environment = copy.deepcopy(manifest.environment)
             self._framework = (
-                copy.deepcopy(self._model_version.manifest.framework.actual_instance)
-                if self._model_version.manifest.framework
+                parse_obj_as(
+                    ModelFrameworkType, manifest.framework.actual_instance.to_dict()
+                )
+                if manifest.framework
                 else None
             )
+            self._tags = copy.deepcopy(self._model_version.manifest.tags) or []
         else:
             self._description = self._model_version.description or ""
-            self._metadata = copy.deepcopy(self._model_version.artifact_metadata)
+            self._metadata = copy.deepcopy(self._model_version.artifact_metadata or {})
             self._environment = None
             self._framework = _ModelFramework.to_model_framework_type(
                 self._model_version.model_framework
             )
+            self._tags = []
 
     def _refetch_model_version(self, reset_mutable_attrs: bool = True):
         _model_version = self._mlfoundry_artifacts_api.get_model_version_get(
@@ -231,13 +237,46 @@ class ModelVersion:
         _validate_artifact_metadata(value)
         self._metadata = copy.deepcopy(value)
 
+    @property
+    def tags(self) -> List[str]:
+        """
+        Get tags for the current artifact version
+        """
+        if not self._model_version.manifest:
+            warnings.warn(
+                message="This model version was created using an older serialization format. tags do not exist, returning empty list",
+                category=DeprecationWarning,
+                stacklevel=2,
+            )
+        return self._tags
+
+    @tags.setter
+    def tags(self, value: List[str]):
+        """
+        Set the tags for current artifact version
+        """
+        if not self._model_version.manifest:
+            warnings.warn(
+                message="This model version was created using an older serialization format. Tags will not be updated",
+                category=DeprecationWarning,
+                stacklevel=2,
+            )
+            return
+        self._tags = copy.deepcopy(value)
+
     @property
     def environment(self) -> Optional[ModelVersionEnvironment]:
         """Get the environment details for the model"""
+        if not self._model_version.manifest:
+            warnings.warn(
+                message="This model version was created using an older serialization format. environment does not exist, returning None",
+                category=DeprecationWarning,
+                stacklevel=2,
+            )
         return self._environment
 
     @environment.setter
-    def environment(self, value: Optional[Dict[str, Any]]):
+    def environment(self, value: Optional[ModelVersionEnvironment]):
         """set the environment details for the model"""
         if not self._model_version.manifest:
             warnings.warn(
@@ -449,6 +488,7 @@ class ModelVersion:
             self._model_version.manifest.framework = (
                 Framework.from_dict(self.framework.dict()) if self.framework else None
             )
+            self._model_version.manifest.tags = self.tags
         try:
             _model_version = self._mlfoundry_artifacts_api.update_model_version_post(
                 update_model_version_request_dto=UpdateModelVersionRequestDto(

{truefoundry-0.5.3rc1 → truefoundry-0.5.3rc2}/truefoundry/ml/model_framework.py

@@ -20,7 +20,8 @@ from truefoundry.common.utils import (
     get_python_version_major_minor,
     list_pip_packages_installed,
 )
-from truefoundry.ml.autogen.client import (
+from truefoundry.ml.autogen.client import (  # type: ignore[attr-defined]
+    ModelVersionEnvironment,
     SklearnSerializationFormat,
     XGBoostSerializationFormat,
 )
@@ -344,7 +345,7 @@ def _fetch_framework_specific_pip_packages(
 
 
 def auto_update_environment_details(
-    environment: autogen_artifacts.ModelVersionEnvironment,
+    environment: ModelVersionEnvironment,
     framework: Optional[ModelFrameworkType],
 ):
     """