-
Notifications
You must be signed in to change notification settings - Fork 889
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
Generate vertex reference docs #8226
Conversation
|
Size Report 1Affected Products
Test Logs |
Size Analysis Report 1This report is too large (1,924,681 characters) to be displayed here in a GitHub comment. Please use the below link to see the full report on Google Cloud Storage.Test Logs |
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.
initial pass done... mostly syntax-fixup requests. Nothing blocking that I can tell from the rendered GitHub docs.
{% endcomment %} | ||
|
||
# FunctionCall interface | ||
A predicted \[FunctionCall\] returned from the model that contains a string representing the \[FunctionDeclaration.name\] and a structured JSON object containing the parameters and their values. |
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.
Do FunctionCall and FunctionDeclaration.name have the proper syntax?
They look like this in the rendered docs: https://screenshot.googleplex.com/Xz3zxfw676YwTkJ.png
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.
Fixed.
{% endcomment %} | ||
|
||
# FunctionDeclaration interface | ||
Structured representation of a function declaration as defined by the \[OpenAPI 3.0 specification\](https://spec.openapis.org/oas/v3.0.3). Included in this declaration are the function name and parameters. This FunctionDeclaration is a representation of a block of code that can be used as a Tool by the model and executed by the client. |
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.
Does OpenAPI 3.0 specification have the proper syntax?
It looks like this in the rendered docs: https://screenshot.googleplex.com/Aw96GRmu59HcZZB
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.
Fixed.
{% endcomment %} | ||
|
||
# FunctionDeclaration interface | ||
Structured representation of a function declaration as defined by the \[OpenAPI 3.0 specification\](https://spec.openapis.org/oas/v3.0.3). Included in this declaration are the function name and parameters. This FunctionDeclaration is a representation of a block of code that can be used as a Tool by the model and executed by the client. |
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.
NOT BLOCKING I think that FunctionDeclaration in the second sentence should be tagged as code.
Note that I've only marked this instance, but it seems there are lots of places throughout the source code that should be tagged as code, like FunctionDeclarationsTool, FunctionCall, ChatSession, generateContentStream(), generateContent(), etc.
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.
Did a pass and tried to catch as many of these as I could.
{% endcomment %} | ||
|
||
# VertexAI interface | ||
An instance of Firebase Vertex AI. |
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.
Should this be?
An instance of Vertex AI for Firebase
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.
Updated.
Co-authored-by: rachelsaunders <52258509+rachelsaunders@users.noreply.github.com>
Co-authored-by: rachelsaunders <52258509+rachelsaunders@users.noreply.github.com>
Autogenerate VertexAI reference docs.
Our doc process is: These markdown files are checked into Github, and once that is done, a separate CL is made to copy them in to the devsite dir in google3. At that point they can be staged and are a little easier to read.
The best way to view these at the moment is probably to go to the entry point page in this branch: https://github.com/firebase/firebase-js-sdk/blob/ch-vertex-docs/docs-devsite/vertexai-preview.md