-
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
Implement standalone chat detection API for text #210
Comments
We’re using the openAI messages object directly, so the I’m going to answer these backwards since I think it makes more sense
We can start off with this for text particularly [I’ve updated the title of the issue, also the standalone detection endpoint we have now is just
Because this is pretty much just a passthrough of message history, we can keep any |
@mdevino the standalone API is mainly targeted to support chat as the input. It doesn't necessarily have to be processed by a chat completion model implementing open-ai API etc. So, parameter wise, we don't need to worry about any of those parameters. If such parameters are supported by the underlying model, then we would basically do a passthrough, as @evaline-ju mentioned. We decided to use "messages" format more closely to open-ai to make it easier for users to pass their already formatted data. |
Description
As a followup to #195, as an orchestrator developer, I want to implement the changes for the updated API, so that I can provide standalone detections on chat. This endpoint is designed for detectors working on full chat histories
Tasks
/text/detection/chat
(accepting chat history request) in orchestratordetection_chat
Acceptance Criteria
The text was updated successfully, but these errors were encountered: