-
Notifications
You must be signed in to change notification settings - Fork 437
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
Refactor rest server implementation #2091
Conversation
9f66ccd
to
4bc35f4
Compare
faa303c
to
d91fb9c
Compare
841d57e
to
26c6a4f
Compare
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.
default: "*args,**kwargs" | ||
RetrievalPrompt: | ||
_path: superduperdb/ext/llm/prompt/RetrievalPrompt | ||
select: |
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.
Here. I think the select field should be enclosed. Otherwise frontend can't render forms.
"select": {
"type": "query",
}
identifier: | ||
type: str | ||
components: | ||
type: [model, listener, vector_index] |
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.
type is used to define what kind of form is rendered. the type of type should be always str
to render in the frontend.
Make changes to make rest compatible with the new serialization protocol.
This greatly simplifies the implementation.