Conversation
|
All commit authors signed the Contributor License Agreement. |
Docs preview deploy✅ Successfully deployed docs preview for commit a267b3a: https://edgedb-docs-k7qcnylht-edgedb.vercel.app (Last updated: Feb 19, 2025, 12:54:35 UTC) |
docs/ai/guide_edgeql.rst
Outdated
| If you'd like to build something on your own, make sure to check out the | ||
| Reference manual in order to learn the details about using different APIs and | ||
| models, configuring prompts or using the UI. Make sure to also check out the | ||
| Gel AI bindings in Python and JavaScript if those languages are relevant to | ||
| you. |
There was a problem hiding this comment.
Let's add links to the reference and other guides
|
|
||
| .. edb:split-section:: | ||
|
|
||
| In this section we'll use |Gel|'s built-in vector search and |
There was a problem hiding this comment.
Let's add some intro to the flashcard app - links to the quickstart, repo, etc.
|
|
||
|
|
||
| .. code-block:: sdl | ||
| :caption: dbschema/default.gel |
There was a problem hiding this comment.
We just changed this btw to be a multi exclusive link from Deck -> Card.
docs/ai/guide_python.rst
Outdated
| "text-embedding-3-small", | ||
| ) | ||
|
|
||
| + gel_client.query( |
There was a problem hiding this comment.
Can we show what a response would look like here?
| accurate results at the cost of time and resource usage when building the | ||
| index. Default: ``100`` | ||
|
|
||
| * ``dimensions``: int64 (Optional) - Embedding dimensions |
There was a problem hiding this comment.
Is there a default here?
docs/ai/reference_extai.rst
Outdated
| * ``OpenAI`` | ||
| * ``Anthropic`` | ||
|
|
||
| ext::ai::ProviderConfig |
There was a problem hiding this comment.
This can be in a follow-up PR, but we should show what custom providers look like somewhere.
| { | ||
| "model": string, // Required: Name of the embedding model | ||
| "inputs": string[], // Required: Array of texts to embed | ||
| "dimensions": number, // Optional: Number of dimensions to truncate to | ||
| "user": string // Optional: User identifier | ||
| } |
There was a problem hiding this comment.
I don't think we do a syntax check pass on JSON, but this isn't valid JSON, so if someone copy-pasted it, it wouldn't work.
Maybe you're meaning to use TypeScript to define this interface? I personally wouldn't bother doing that, and instead just list the properties like you do below.
| AI client classes | ||
| ----------------- | ||
|
|
||
| Core classes |
There was a problem hiding this comment.
Didn't we recently rename these classes?
Co-authored-by: Scott Trinh <scott@edgedb.com>
Co-authored-by: Scott Trinh <scott@edgedb.com>
- Convert type descriptions to blocks - Fill in UI section
Co-authored-by: Scott Trinh <scott@edgedb.com>
Co-authored-by: Scott Trinh <scott@edgedb.com>
3 participants
No description provided.