Skip to content

Commit

Permalink
fix: enable_session_persistence in AgentConfig should be optional (me…
Browse files Browse the repository at this point in the history
…ta-llama#1012)

# What does this PR do?
This issue was discovered in
meta-llama#1009 (comment).

## Test Plan

This field is no longer required after the change.

[//]: # (## Documentation)
[//]: # (- [ ] Added a Changelog entry if the change is significant)

---------

Signed-off-by: Yuan Tang <[email protected]>
Co-authored-by: Ashwin Bharambe <[email protected]>
  • Loading branch information
2 people authored and srikanthbachala20 committed Feb 27, 2025
1 parent 15d4604 commit cd73d39
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions docs/_static/llama-stack-spec.html
Original file line number Diff line number Diff line change
Expand Up @@ -2724,7 +2724,8 @@
"type": "string"
},
"enable_session_persistence": {
"type": "boolean"
"type": "boolean",
"default": false
},
"response_format": {
"$ref": "#/components/schemas/ResponseFormat"
Expand All @@ -2733,8 +2734,7 @@
"additionalProperties": false,
"required": [
"model",
"instructions",
"enable_session_persistence"
"instructions"
]
},
"AgentTool": {
Expand Down
2 changes: 1 addition & 1 deletion docs/_static/llama-stack-spec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -1660,13 +1660,13 @@ components:
type: string
enable_session_persistence:
type: boolean
default: false
response_format:
$ref: '#/components/schemas/ResponseFormat'
additionalProperties: false
required:
- model
- instructions
- enable_session_persistence
AgentTool:
oneOf:
- type: string
Expand Down
2 changes: 1 addition & 1 deletion llama_stack/apis/agents/agents.py
Original file line number Diff line number Diff line change
Expand Up @@ -179,7 +179,7 @@ def model_post_init(self, __context):
class AgentConfig(AgentConfigCommon):
model: str
instructions: str
enable_session_persistence: bool
enable_session_persistence: Optional[bool] = False
response_format: Optional[ResponseFormat] = None


Expand Down

0 comments on commit cd73d39

Please sign in to comment.