-
Notifications
You must be signed in to change notification settings - Fork 15
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
Assistants YAML config #643
Conversation
It looks like only the JVM target is supported charleskorn/kaml#232. We may need to do further research to find an alternative for multiplatform. |
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.
🚀 LGTM!
Just a minor comment, but maybe it's not important for now
core/src/commonMain/kotlin/com/xebia/functional/xef/llm/assistants/Assistant.kt
Show resolved
Hide resolved
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.
Couple of suggestions to fix the compilation issues after adding the toolsConfig
arg.
core/src/commonMain/kotlin/com/xebia/functional/xef/llm/assistants/Assistant.kt
Outdated
Show resolved
Hide resolved
core/src/commonMain/kotlin/com/xebia/functional/xef/llm/assistants/Assistant.kt
Outdated
Show resolved
Hide resolved
…ants/Assistant.kt Co-authored-by: David Vega Lichacz <[email protected]>
…ants/Assistant.kt Co-authored-by: David Vega Lichacz <[email protected]>
Given some yaml config such as:
Creates or updates an Open AI assistant.
Currently metadata is being ignored until we find the best way to represent it. Current value is
null
.