From c29fa56ddebdc8c3ca1abff042e21b6c999311d3 Mon Sep 17 00:00:00 2001 From: Xi Yan Date: Wed, 13 Nov 2024 10:44:39 -0500 Subject: [PATCH] add inline:: prefix for localfs provider (#441) # What does this PR do? - add inline:: prefix for localfs provider ## Test Plan ``` llama stack run datasetio: - provider_id: localfs-0 provider_type: inline::localfs config: {} ``` ``` pytest -v -s -m meta_reference_eval_fireworks_inference eval/test_eval.py pytest -v -s -m localfs datasetio/test_datasetio.py ``` ## Sources Please link relevant resources if necessary. ## Before submitting - [ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case). - [ ] Ran pre-commit to handle lint / formatting issues. - [ ] Read the [contributor guideline](https://github.com/meta-llama/llama-stack/blob/main/CONTRIBUTING.md), Pull Request section? - [ ] Updated relevant documentation. - [ ] Wrote necessary unit or integration tests. --- llama_stack/providers/registry/datasetio.py | 2 +- llama_stack/providers/tests/datasetio/fixtures.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/llama_stack/providers/registry/datasetio.py b/llama_stack/providers/registry/datasetio.py index 2d1c722f0..7893bcde4 100644 --- a/llama_stack/providers/registry/datasetio.py +++ b/llama_stack/providers/registry/datasetio.py @@ -13,7 +13,7 @@ def available_providers() -> List[ProviderSpec]: return [ InlineProviderSpec( api=Api.datasetio, - provider_type="localfs", + provider_type="inline::localfs", pip_packages=["pandas"], module="llama_stack.providers.inline.datasetio.localfs", config_class="llama_stack.providers.inline.datasetio.localfs.LocalFSDatasetIOConfig", diff --git a/llama_stack/providers/tests/datasetio/fixtures.py b/llama_stack/providers/tests/datasetio/fixtures.py index 60f89de46..f0c8cbbe1 100644 --- a/llama_stack/providers/tests/datasetio/fixtures.py +++ b/llama_stack/providers/tests/datasetio/fixtures.py @@ -24,7 +24,7 @@ def datasetio_localfs() -> ProviderFixture: providers=[ Provider( provider_id="localfs", - provider_type="localfs", + provider_type="inline::localfs", config={}, ) ],