Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions runware/types.py
Original file line number Diff line number Diff line change
Expand Up @@ -888,6 +888,8 @@ class ISettings(SerializableMixin):
languageBoost: Optional[str] = None
turbo: Optional[bool] = None
lyrics: Optional[str] = None
instrumental: Optional[bool] = None
lyricsOptimizer: Optional[bool] = None
guidanceType: Optional[str] = None
textNormalization: Optional[bool] = None
topK: Optional[int] = None
Copy link

Copilot AI Apr 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ISettings defines topK twice (here in the Audio section and again later in the Text section). In a dataclass/class body, the later definition overwrites the earlier one, so the audio topK value can never be set/serialized as intended. Consider renaming these fields to be modality-specific (e.g., audioTopK/textTopK) or splitting settings into nested structs to avoid silent overrides (same issue also exists for other duplicated names like thinking).

Suggested change
topK: Optional[int] = None
audioTopK: Optional[int] = None

Copilot uses AI. Check for mistakes.
Expand Down Expand Up @@ -1831,6 +1833,7 @@ class IAudioInference:
taskUUID: Optional[str] = None
outputType: Optional[IOutputType] = None
outputFormat: Optional[IAudioOutputFormat] = None
outputQuality: Optional[int] = None
audioSettings: Optional[IAudioSettings] = None
includeCost: Optional[bool] = None
numberResults: Optional[int] = 1
Expand Down