-
Notifications
You must be signed in to change notification settings - Fork 119
chore(eng-docs): add docs for knowledge gaps #3976
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
Changes from 6 commits
b1faa09
107c7c3
face769
c9a1189
2fd59ae
495a519
a725b3b
c9bd6e9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,42 @@ | ||
# Generating Embeddings | ||
|
||
Embeddings are what is used for our [Doc Search](./docsearch.md) functionality. OpenAI embeddings are a technique that uses machine learning and big data to convert unstructured data into structured vector spaces. | ||
|
||
In our use case it converts plain text such as search criteria, mdx headers and GitHub discussion titles. It uses the model `text-embedding-ada-002` and outputs a similar structure to: `[-0.005330325,0.018767769,0.00020701668,-0.0011101937, ...]` | ||
|
||
## Local Development | ||
|
||
In order to develop locally you will need to startup a local instance of Supabase. The code for this is found in `/apps/backend`. Follow [this](../../../apps/backend/README.md) document to get setup. | ||
|
||
After you have it setup you should be able to access Supabase at: http://127.0.0.1:54323. If you have no tables, you have not applied migrations. You can run ```yarn workspace @twilio-paste/backend db:reset``` from the root of the project. | ||
|
||
**Note**: if you see an error for vector packages go into [20230928013336_initial_schema](../../../apps/backend/supabase/migrations/20230928013336_initial_schema.sql) and change the following **without committing**: | ||
|
||
```sql | ||
create extension if not exists "vector" with schema "public" version '0.5.0'; | ||
/* to */ | ||
create extension if not exists "vector" with schema "public"; | ||
``` | ||
|
||
### Environment Variables | ||
|
||
In order to do any GH action or assistant development on the site you will need to set environment variables in ```packages/paste-website/.env```. | ||
|
||
``` | ||
OPENAI_API_KEY="" // USE YOUR PERSONALTOKEN FOR LOCAL DEV | ||
SUPABASE_URL="http://127.0.0.1:54321" // PRINTED TO CONSOLE AFTER STARTING CONTAINER | ||
SUPABASE_KEY="" // PRINTED TO CONSOLE AFTER STARTING CONTAINER | ||
GH_SERVICE_ACC_DISCUSSIONS_TOKEN="" // IN 1Password UNDER github.com ENTRY | ||
``` | ||
|
||
### Generating Data | ||
|
||
The best way to generate data is to run the nightly embed script `generate:embeddings`. This will update the tables: `page` and `page_section`. | ||
|
||
## Table Structure | ||
|
||
While there are other tables the only ones that concern the embeddings creation are: | ||
- **page**: Stores the metadata of the entry. Key columns are the checksum (used to determine whether to update the record), path (either the url of the page or the github discussion), type (github-discussion or markdown) | ||
- **page_sections**: contains the search embeddings. Key columns are content (plain text headings/titles), embedding (the vector spaces created from OpenAI), slug (yo string of content or the discussion/answer in GitHub). | ||
|
||
Both tables are related with page being thge parent. They are joined by `page.id on page_section.page_id`. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [misspell] reported by reviewdog 🐶 |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
# Publishing NPM Package | ||
|
||
Paste core uses [changesets](https://github.com/changesets/changesets) to manage versions and changelogs. It has great support for mono-repos and multipackage repositories, ideal for `@twilio-paste/core`. | ||
|
||
Changesets has a great [GitHub action](https://github.com/changesets/action) that will manage the release by creating a PR, periodically pulling changes from main. No code is published to NPM until this PR is merged, which is controlled by the team. | ||
|
||
The PR will always be called `Version Packages` and lists all the changes that have been made since the last release. The description will also update with the entries in the changesets from the PRs merged to easily see what will be getting released. | ||
|
||
There is a step in the GitHub Action [on_merge_to_main](../../.github/workflows/on_merge_to_main.yml) with the name `Create Pull Request or Publish to npm`. This defines commands to run from [package.json](../../package.json) for what operation. | ||
|
||
- version: this removes all of the temporary changeset files which are generated during development. It aggregates them all to a changelog entry. | ||
- publish: responsible for publishing the package to NPM. | ||
- commit: "chore(release): version packages" the commit message on squash and merge. |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -81,6 +81,11 @@ module.exports = function (plop) { | |
path: "packages/paste-core/{{component-type}}/{{kebabCase component-name}}/tsconfig.json", | ||
templateFile: "tools/plop-templates/tsconfig.hbs", | ||
}, | ||
{ | ||
type: "add", | ||
path: "packages/paste-core/{{component-type}}/{{kebabCase component-name}}/CHANGELOG.md", | ||
templateFile: "tools/plop-templates/CHANGELOG.hbs", | ||
}, | ||
Comment on lines
+84
to
+88
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is a mandatory file that is needed when developing the website section for new components. Good to add it in at gen level to stop getting caught out and manually having to update the components package later. |
||
], | ||
}); | ||
}; |
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.
[misspell] reported by reviewdog 🐶
"refernce" is a misspelling of "references"