-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Conversation
✅ Deploy Preview for continuedev canceled.
|
@@ -157,7 +157,8 @@ export default class DocsService { | |||
|
|||
const currentStatus = this.statuses.get(doc.startUrl); | |||
if (currentStatus) { | |||
return currentStatus; | |||
this.handleStatusUpdate(currentStatus); |
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.
stuck at zero fix
// Since SiteIndexingConfig is likely to be valid | ||
try { | ||
await provider.embed(["continue-test-run"]); | ||
} catch (e) { |
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.
embedder test run
// TODO - this might significantly impact performance during indexing | ||
const sortedConfigDocs = useMemo(() => { | ||
const sorter = (status: IndexingStatus["status"]) => { | ||
// TODO - further sorting? |
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.
sort by indexing status in More page
Description
indexAndAdd
a bit to account for it