-
Notifications
You must be signed in to change notification settings - Fork 89
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
Upgrade to LangChain4j 0.36.2 #1097
Conversation
jmartisk
commented
Nov 22, 2024
•
edited by geoand
Loading
edited by geoand
- Closes: Update to LangChain4j 0.36 #1075
@@ -1,7 +1,5 @@ | |||
package io.quarkiverse.langchain4j.runtime.aiservice; | |||
|
|||
import static org.apache.commons.lang3.StringUtils.EMPTY; |
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.
This dependency has somehow disappeared, but I don't see why we should re-add it just for a constant of an empty string
This comment has been minimized.
This comment has been minimized.
b459454
to
3168cce
Compare
Status for workflow
|
@@ -116,6 +118,12 @@ public MistralAiEmbeddingResponse embedding(MistralAiEmbeddingRequest request) { | |||
return restApi.embedding(request, apiKey); | |||
} | |||
|
|||
// TODO: we don't provide support for MistralAiModerationModel yet |
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.
We don't?
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.
Well, users can instantiate it directly, but we don't have a declarative way via config properties for it yet. I plan to add that via #1095
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.
Gotcha, thanks
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.
Great stuff @jmartisk!