-
Notifications
You must be signed in to change notification settings - Fork 1.2k
fix: enable_session_persistence in AgentConfig should be optional #1012
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
docs/_static/llama-stack-spec.html
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just checking -- did you update this using cd docs/openapi_generator && sh run_openapi_generator.sh
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I removed the unrelated change (accidentally included by my other work)
d959ad5
to
033182e
Compare
Signed-off-by: Yuan Tang <terrytangyuan@gmail.com>
@terrytangyuan I just rebased and resolved the doc generation conflict |
Thank you! |
…ev (#145) # What does this PR do? - adapt to llamastack/llama-stack#1012 [//]: # (If resolving an issue, uncomment and update the line below) [//]: # (Closes #[issue-number]) ## Test Plan [Describe the tests you ran to verify your changes with result summaries. *Provide clear instructions so the plan can be easily re-executed.*] [//]: # (## Documentation) [//]: # (- [ ] Added a Changelog entry if the change is significant)
What does this PR do?
This issue was discovered in #1009 (comment).
Test Plan
This field is no longer required after the change.