Skip to content

Comments

feat(core): improve diagnostics for ingestion and embedding#381

Open
BloggerBust wants to merge 1 commit intomorphik-org:mainfrom
BloggerBust:fix/issue-340-ingestion-embedding-diagnostics
Open

feat(core): improve diagnostics for ingestion and embedding#381
BloggerBust wants to merge 1 commit intomorphik-org:mainfrom
BloggerBust:fix/issue-340-ingestion-embedding-diagnostics

Conversation

@BloggerBust
Copy link

Ref: #340

Changes

  • Ingestion: warn when parsing yields empty text in cases where text is expected
  • Ingestion: include diagnostic context for the error raised when there are no no content chunks
  • Embedding: enrich LiteLLM embedding error logs with model_key, model_name, api_base, num_texts context

Tests

  • Added unit tests for ingestion diagnostics helpers
  • Added unit tests for LiteLLM embedding error logging

Notes

  • Follow-up PR planned for completion side LiteLLM diagnostics to keep this PR focused.

…rage

- warn on empty parsing when text is expected
- include context in ingestion diagnostics "no content chunks" error
- enrich LiteLLM embedding error logs with model context
- add unit tests for diagnostics helpers and embedding error logging
@CLAassistant
Copy link

CLAassistant commented Feb 16, 2026

CLA assistant check
All committers have signed the CLA.

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