diff --git a/modelscope/pipelines/builder.py b/modelscope/pipelines/builder.py index 58ec4db5..097ff9ee 100644 --- a/modelscope/pipelines/builder.py +++ b/modelscope/pipelines/builder.py @@ -310,9 +310,6 @@ def pipeline(task: str = None, model[0], revision=model_revision) check_config(cfg) pipeline_name = cfg.pipeline.type - else: - # used for test case, when model is str and is not hub path - pipeline_name = get_pipeline_by_model_name(task, model) elif model is not None: # get pipeline info from Model object first_model = model[0] if isinstance(model, list) else model @@ -375,19 +372,3 @@ def get_default_pipeline_info(task): else: pipeline_name, default_model = DEFAULT_MODEL_FOR_PIPELINE[task] return pipeline_name, default_model - - -def get_pipeline_by_model_name(task: str, model: Union[str, List[str]]): - """ Get pipeline name by task name and model name - - Args: - task (str): task name. - model (str| list[str]): model names - """ - if isinstance(model, str): - model_key = model - else: - model_key = '_'.join(model) - assert model_key in PIPELINES.modules[task], \ - f'pipeline for task {task} model {model_key} not found.' - return model_key diff --git a/modelscope/pipelines/util.py b/modelscope/pipelines/util.py index 2c2c7751..99a11317 100644 --- a/modelscope/pipelines/util.py +++ b/modelscope/pipelines/util.py @@ -35,8 +35,7 @@ def is_official_hub_path(path: Union[str, List], _ = HubApi().get_model(path, revision=revision) return True except Exception as e: - logger.warning(f'get model exception: {e}') - return False + raise ValueError(f'invalid model repo path {e}') if isinstance(path, str): return is_official_hub_impl(path)