Removes pipleine tag validation.

Author: mrDzurb

ads/aqua/extension/model_handler.py

@@ -13,7 +13,6 @@
 from ads.aqua.extension.base_handler import AquaAPIhandler
 from ads.aqua.extension.errors import Errors
 from ads.aqua.model import AquaModelApp
-from ads.aqua.model.constants import ModelTask
 from ads.aqua.model.entities import AquaModelSummary, HFModelSummary
 from ads.aqua.ui import ModelFormat
 
@@ -230,16 +229,17 @@ def post(self, *args, **kwargs):
                 "Please verify the model's status on the Hugging Face Model Hub or select a different model."
             )
 
-        # Check pipeline_tag, it should be `text-generation`
-        if not (
-            hf_model_info.pipeline_tag
-            and hf_model_info.pipeline_tag.lower() in ModelTask
-        ):
-            raise AquaRuntimeError(
-                f"Unsupported pipeline tag for the chosen model: '{hf_model_info.pipeline_tag}'. "
-                f"AQUA currently supports the following tasks only: {', '.join(ModelTask.values())}. "
-                "Please select a model with a compatible pipeline tag."
-            )
+        # Commented the validation below to let users to register any model type.
+        # # Check pipeline_tag, it should be `text-generation`
+        # if not (
+        #     hf_model_info.pipeline_tag
+        #     and hf_model_info.pipeline_tag.lower() in ModelTask
+        # ):
+        #     raise AquaRuntimeError(
+        #         f"Unsupported pipeline tag for the chosen model: '{hf_model_info.pipeline_tag}'. "
+        #         f"AQUA currently supports the following tasks only: {', '.join(ModelTask.values())}. "
+        #         "Please select a model with a compatible pipeline tag."
+        #     )
 
         # Check if it is a service/verified model
         aqua_model_info: AquaModelSummary = self._find_matching_aqua_model(

tests/unitary/with_extras/aqua/test_model_handler.py

@@ -264,22 +264,22 @@ def test_post_negative(self, mock_uuid, mock_format_hf_custom_error_message):
             )
         get_hf_model_info.cache_clear()
 
-        # case 6
-        self.mock_handler.get_json_body = MagicMock(
-            return_value={"model_id": "test_model_id"}
-        )
-        with patch.object(HfApi, "model_info") as mock_model_info:
-            mock_model_info.return_value = MagicMock(
-                disabled=False, id="test_model_id", pipeline_tag="not-text-generation"
-            )
-            self.mock_handler.post()
-            self.mock_handler.finish.assert_called_with(
-                '{"status": 400, "message": "Something went wrong with your request.", '
-                '"service_payload": {}, "reason": "Unsupported pipeline tag for the chosen '
-                "model: 'not-text-generation'. AQUA currently supports the following tasks only: "
-                f'{", ".join(ModelTask.values())}. '
-                'Please select a model with a compatible pipeline tag.", "request_id": "###"}'
-            )
+        # # case 6 pipeline Tag
+        # self.mock_handler.get_json_body = MagicMock(
+        #     return_value={"model_id": "test_model_id"}
+        # )
+        # with patch.object(HfApi, "model_info") as mock_model_info:
+        #     mock_model_info.return_value = MagicMock(
+        #         disabled=False, id="test_model_id", pipeline_tag="not-text-generation"
+        #     )
+        #     self.mock_handler.post()
+        #     self.mock_handler.finish.assert_called_with(
+        #         '{"status": 400, "message": "Something went wrong with your request.", '
+        #         '"service_payload": {}, "reason": "Unsupported pipeline tag for the chosen '
+        #         "model: 'not-text-generation'. AQUA currently supports the following tasks only: "
+        #         f'{", ".join(ModelTask.values())}. '
+        #         'Please select a model with a compatible pipeline tag.", "request_id": "###"}'
+        #     )
         get_hf_model_info.cache_clear()
 
     @patch("uuid.uuid4")