Prefer Service.Settings over raw settings class names across all services
Replace direct references to settings class names (e.g. `FooSettings`) with the nested `Settings` alias form throughout all 87 service files: - Type annotations: `Settings` - Runtime code: `self.Settings` - Docstrings: `ServiceClass.Settings` - Cross-file inheritance: `ParentService.Settings` This makes the `Settings` alias the canonical way to reference a service's settings, keeping only the class definition and alias assignment as the remaining hits for each raw settings class name.
This commit is contained in:
@@ -12,13 +12,13 @@ from typing import Optional
|
||||
from loguru import logger
|
||||
|
||||
from pipecat.adapters.services.open_ai_adapter import OpenAILLMInvocationParams
|
||||
from pipecat.services.openai.base_llm import OpenAILLMSettings
|
||||
from pipecat.services.openai.base_llm import BaseOpenAILLMService
|
||||
from pipecat.services.openai.llm import OpenAILLMService
|
||||
from pipecat.services.settings import _warn_deprecated_param
|
||||
|
||||
|
||||
@dataclass
|
||||
class CerebrasLLMSettings(OpenAILLMSettings):
|
||||
class CerebrasLLMSettings(BaseOpenAILLMService.Settings):
|
||||
"""Settings for CerebrasLLMService."""
|
||||
|
||||
pass
|
||||
@@ -32,7 +32,7 @@ class CerebrasLLMService(OpenAILLMService):
|
||||
"""
|
||||
|
||||
Settings = CerebrasLLMSettings
|
||||
_settings: CerebrasLLMSettings
|
||||
_settings: Settings
|
||||
|
||||
def __init__(
|
||||
self,
|
||||
@@ -40,7 +40,7 @@ class CerebrasLLMService(OpenAILLMService):
|
||||
api_key: str,
|
||||
base_url: str = "https://api.cerebras.ai/v1",
|
||||
model: Optional[str] = None,
|
||||
settings: Optional[CerebrasLLMSettings] = None,
|
||||
settings: Optional[Settings] = None,
|
||||
**kwargs,
|
||||
):
|
||||
"""Initialize the Cerebras LLM service.
|
||||
@@ -51,18 +51,18 @@ class CerebrasLLMService(OpenAILLMService):
|
||||
model: The model identifier to use. Defaults to "gpt-oss-120b".
|
||||
|
||||
.. deprecated:: 0.0.105
|
||||
Use ``settings=OpenAILLMSettings(model=...)`` instead.
|
||||
Use ``settings=CerebrasLLMService.Settings(model=...)`` instead.
|
||||
|
||||
settings: Runtime-updatable settings. When provided alongside deprecated
|
||||
parameters, ``settings`` values take precedence.
|
||||
**kwargs: Additional keyword arguments passed to OpenAILLMService.
|
||||
"""
|
||||
# 1. Initialize default_settings with hardcoded defaults
|
||||
default_settings = CerebrasLLMSettings(model="gpt-oss-120b")
|
||||
default_settings = self.Settings(model="gpt-oss-120b")
|
||||
|
||||
# 2. Apply direct init arg overrides (deprecated)
|
||||
if model is not None:
|
||||
_warn_deprecated_param("model", CerebrasLLMSettings, "model")
|
||||
_warn_deprecated_param("model", self.Settings, "model")
|
||||
default_settings.model = model
|
||||
|
||||
# 3. (No step 3, as there's no params object to apply)
|
||||
|
||||
Reference in New Issue
Block a user