Skip to content

Docs Fixes and Embeddings Provider Test Run #4170

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

Merged
merged 8 commits into from
Feb 15, 2025
Merged

Conversation

RomneyDa
Copy link
Collaborator

@RomneyDa RomneyDa commented Feb 15, 2025

Description

  • Add an embeddings test run to the beginning of docs indexing, and rearrange indexAndAdd a bit to account for it
  • Fix docs status initialized to zero
  • Sort docs by indexing status in More page

Copy link

netlify bot commented Feb 15, 2025

Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit 06166cb
🔍 Latest deploy log https://app.netlify.com/sites/continuedev/deploys/67b02a6127c7430008ceb065

@RomneyDa RomneyDa changed the title Test Run Embeddings Provider prior to indexing Docs Fixes and Embeddings Provider Test Run Feb 15, 2025
@@ -157,7 +157,8 @@ export default class DocsService {

const currentStatus = this.statuses.get(doc.startUrl);
if (currentStatus) {
return currentStatus;
this.handleStatusUpdate(currentStatus);
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

stuck at zero fix

// Since SiteIndexingConfig is likely to be valid
try {
await provider.embed(["continue-test-run"]);
} catch (e) {
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

embedder test run

// TODO - this might significantly impact performance during indexing
const sortedConfigDocs = useMemo(() => {
const sorter = (status: IndexingStatus["status"]) => {
// TODO - further sorting?
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sort by indexing status in More page

@sestinj sestinj merged commit 6c8f035 into main Feb 15, 2025
31 checks passed
@sestinj sestinj deleted the dallin/docs-issues branch February 15, 2025 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants