From 0b6a7abb280ee5f07f9cc8c4a36fd47733a53281 Mon Sep 17 00:00:00 2001 From: Ashwin Bharambe Date: Tue, 5 Aug 2025 14:36:08 -0700 Subject: [PATCH] more fixes --- docs/source/providers/agents/index.md | 2 +- docs/source/providers/datasetio/index.md | 2 +- docs/source/providers/eval/index.md | 2 +- docs/source/providers/files/index.md | 2 +- docs/source/providers/inference/index.md | 2 +- docs/source/providers/post_training/index.md | 2 +- docs/source/providers/post_training/inline_huggingface.md | 2 +- docs/source/providers/safety/index.md | 2 +- docs/source/providers/scoring/index.md | 2 +- docs/source/providers/telemetry/index.md | 2 +- docs/source/providers/tool_runtime/index.md | 2 +- docs/source/providers/vector_io/index.md | 2 +- .../providers/inline/post_training/huggingface/config.py | 2 +- scripts/provider_codegen.py | 2 +- tests/integration/agents/test_openai_responses.py | 5 ++++- 15 files changed, 18 insertions(+), 15 deletions(-) diff --git a/docs/source/providers/agents/index.md b/docs/source/providers/agents/index.md index a88f085ad..92bf9edc0 100644 --- a/docs/source/providers/agents/index.md +++ b/docs/source/providers/agents/index.md @@ -1,4 +1,4 @@ -# Agents +# Agents ## Overview diff --git a/docs/source/providers/datasetio/index.md b/docs/source/providers/datasetio/index.md index 9b0f385f4..94a97e2ed 100644 --- a/docs/source/providers/datasetio/index.md +++ b/docs/source/providers/datasetio/index.md @@ -1,4 +1,4 @@ -# Datasetio +# Datasetio ## Overview diff --git a/docs/source/providers/eval/index.md b/docs/source/providers/eval/index.md index f8d24a820..d180d256c 100644 --- a/docs/source/providers/eval/index.md +++ b/docs/source/providers/eval/index.md @@ -1,4 +1,4 @@ -# Eval +# Eval ## Overview diff --git a/docs/source/providers/files/index.md b/docs/source/providers/files/index.md index 8d4f8773a..692aad3ca 100644 --- a/docs/source/providers/files/index.md +++ b/docs/source/providers/files/index.md @@ -1,4 +1,4 @@ -# Files +# Files ## Overview diff --git a/docs/source/providers/inference/index.md b/docs/source/providers/inference/index.md index 207c28c64..1c7bc86b9 100644 --- a/docs/source/providers/inference/index.md +++ b/docs/source/providers/inference/index.md @@ -1,4 +1,4 @@ -# Inference +# Inference ## Overview diff --git a/docs/source/providers/post_training/index.md b/docs/source/providers/post_training/index.md index fb6af2d57..c6c92c40e 100644 --- a/docs/source/providers/post_training/index.md +++ b/docs/source/providers/post_training/index.md @@ -1,4 +1,4 @@ -# Post_Training +# Post_Training ## Overview diff --git a/docs/source/providers/post_training/inline_huggingface.md b/docs/source/providers/post_training/inline_huggingface.md index 0a8745e71..800bae47a 100644 --- a/docs/source/providers/post_training/inline_huggingface.md +++ b/docs/source/providers/post_training/inline_huggingface.md @@ -27,7 +27,7 @@ HuggingFace-based post-training provider for fine-tuning models using the Huggin | `dpo_beta` | `` | No | 0.1 | | | `use_reference_model` | `` | No | True | | | `dpo_loss_type` | `Literal['sigmoid', 'hinge', 'ipo', 'kto_pair'` | No | sigmoid | | -| `dpo_output_dir` | `` | No | ./checkpoints/dpo | | +| `dpo_output_dir` | `` | No | /var/folders/mj/t_stv1ys7637vqf2_b4yf67m0000gn/T/dpo_output_q24jwy0r | | ## Sample Configuration diff --git a/docs/source/providers/safety/index.md b/docs/source/providers/safety/index.md index f82694ac8..5ddda2242 100644 --- a/docs/source/providers/safety/index.md +++ b/docs/source/providers/safety/index.md @@ -1,4 +1,4 @@ -# Safety +# Safety ## Overview diff --git a/docs/source/providers/scoring/index.md b/docs/source/providers/scoring/index.md index 31a87c555..f3bd48eb0 100644 --- a/docs/source/providers/scoring/index.md +++ b/docs/source/providers/scoring/index.md @@ -1,4 +1,4 @@ -# Scoring +# Scoring ## Overview diff --git a/docs/source/providers/telemetry/index.md b/docs/source/providers/telemetry/index.md index 2451e8f62..c7fbfed73 100644 --- a/docs/source/providers/telemetry/index.md +++ b/docs/source/providers/telemetry/index.md @@ -1,4 +1,4 @@ -# Telemetry +# Telemetry ## Overview diff --git a/docs/source/providers/tool_runtime/index.md b/docs/source/providers/tool_runtime/index.md index a0b835e3b..8d29aed43 100644 --- a/docs/source/providers/tool_runtime/index.md +++ b/docs/source/providers/tool_runtime/index.md @@ -1,4 +1,4 @@ -# Tool_Runtime +# Tool_Runtime ## Overview diff --git a/docs/source/providers/vector_io/index.md b/docs/source/providers/vector_io/index.md index a7703ae14..28ae523d7 100644 --- a/docs/source/providers/vector_io/index.md +++ b/docs/source/providers/vector_io/index.md @@ -1,4 +1,4 @@ -# Vector_Io +# Vector_Io ## Overview diff --git a/llama_stack/providers/inline/post_training/huggingface/config.py b/llama_stack/providers/inline/post_training/huggingface/config.py index 52d3d0a76..07b76b3c5 100644 --- a/llama_stack/providers/inline/post_training/huggingface/config.py +++ b/llama_stack/providers/inline/post_training/huggingface/config.py @@ -4,10 +4,10 @@ # This source code is licensed under the terms described in the LICENSE file in # the root directory of this source tree. +import tempfile from typing import Any, Literal from pydantic import BaseModel, Field -import tempfile class HuggingFacePostTrainingConfig(BaseModel): diff --git a/scripts/provider_codegen.py b/scripts/provider_codegen.py index 80c5b7840..f770c264d 100755 --- a/scripts/provider_codegen.py +++ b/scripts/provider_codegen.py @@ -255,7 +255,7 @@ def process_provider_registry(progress, change_tracker: ChangedPathTracker) -> N change_tracker.add_paths(doc_output_dir) index_content = [] - index_content.append(f"# {api_name.title()} \n") + index_content.append(f"# {api_name.title()}\n") index_content.append("## Overview\n") index_content.append( diff --git a/tests/integration/agents/test_openai_responses.py b/tests/integration/agents/test_openai_responses.py index bb838d921..c783cf99b 100644 --- a/tests/integration/agents/test_openai_responses.py +++ b/tests/integration/agents/test_openai_responses.py @@ -4,7 +4,7 @@ # This source code is licensed under the terms described in the LICENSE file in # the root directory of this source tree. import pytest -from openai import BadRequestError +from openai import BadRequestError, OpenAI from llama_stack.core.library_client import LlamaStackAsLibraryClient @@ -36,6 +36,9 @@ from llama_stack.core.library_client import LlamaStackAsLibraryClient ], ) def test_responses_store(compat_client, text_model_id, stream, tools): + if not isinstance(compat_client, OpenAI): + pytest.skip("OpenAI client is required until responses.delete() exists in llama-stack-client") + message = "What's the weather in Tokyo?" + ( " YOU MUST USE THE get_weather function to get the weather." if tools else "" )