Added Seed in ModelParams for Summarization Requests #39
Conversation
…2002a42f58a3bd8a8e20).
…251ea09ca6f91820f5a4).
…c8b09fb4cbef80275fd0).
…81a5566f182fa331da13).
…288bf119f202e14786a0).
|
I don't know what's up with permission changes to these proto files. |
There was a problem hiding this comment.
I don't know what's up with these proto file permission changes.
From my side, this is good. @pwoznic whenever you're ready to make this public, please approve and merge. Thanks.
serving.proto
Outdated
| } | ||
| ModelParams model_params = 215; | ||
|
|
||
| // Parameters for the citation style. | ||
| CitationParams citation_params = 220; |
There was a problem hiding this comment.
Not all prompts support citation prompts. @pwoznic we should have a list somewhere in the docs that lists which prompts support this.
There was a problem hiding this comment.
The citations content lists the supposed prompts (5 at the moment). Edit: Just noticed the date. I think it was addressed in Slack. Was about to merge but just saw a conflict.
admin_apikey.proto
Outdated
| @@ -47,6 +47,8 @@ message ApiKey { | |||
| optional int64 ts_end = 6; | |||
| // Status of the ApiKey. | |||
| ApiKeyStatus status = 7; | |||
| // An id of the key that is not sensitive. | |||
| uint32 public_id = 8; | |||
There was a problem hiding this comment.
@nikron are you okay making this part of the public api? i don't see a reason why not, but just FYI, this is going public.
Heads up!
These protobufs in this repository are automatically generated from Vectara and
the repository is read-only for the general public. As a result, we do not
accept pull requests to this GitHub repository.
If you have feedback on Vectara or its APIs, please let us know in our
forums: https://discuss.vectara.com/