A-star-logic / memoire

Other
1 stars 0 forks source link

Resolve cohere model error from Sentry #42

Closed sriram-vinnakota closed 2 weeks ago

sriram-vinnakota commented 2 weeks ago

Hi,

The error in #38 is regarding character limit to cohere embedding v3 model, token limit also still applicable.

The charecter limit is the estimate they provided for 512 tokes is 2048 characters.

so I have added the character limit on top of token limit.

close #38

github-actions[bot] commented 2 weeks ago

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 45.64% 1205 / 2640
🔵 Statements 45.64% 1205 / 2640
🔵 Functions 82.5% 66 / 80
🔵 Branches 76.68% 171 / 223
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/ai/embedding/ai-embeddings-interface.ts 6.25% 100% 0% 6.25% 14-37
src/ai/embedding/chunking/ai-chunking-fixed-size.ts 100% 100% 100% 100%
src/ai/embedding/model/ai-embedding-model-cohere.ts 85.36% 69.23% 100% 85.36% 110-111, 124-127, 145-146, 159-162
src/api/search/api-search-routes.ts 100% 100% 100% 100%
src/api/search/api-search-schemas.ts 100% 100% 100% 100%
Generated in workflow #72 for commit 05bc041 by the Vitest Coverage Report Action