Skip to content

Conversation

@tonykipkemboi
Copy link
Member

Summary

  • Replace config.embedder with config.embedding_model in docs.
  • Mark vectordb as required; add commented examples for Chroma Settings and Qdrant VectorParams.
  • Add provider-specific commented examples (OpenAI, Google, Cohere, Ollama).
  • Update English, Korean, and Portuguese File & Document tool pages; minor KO/PT-BR Datadog tweaks.
  • Aligns docs with current RagTool config parsing.

Motivation

  • Tools now read embedding configuration via embedding_model and support vectordb configs directly; old embedder examples are misleading.

Scope of changes

  • en: PDF/TXT/XML/MDX/DOCX/CSV/Directory
  • ko: PDF/TXT/XML/MDX/DOCX/CSV/Directory
  • pt-BR: PDF/TXT/XML/MDX/DOCX/Directory
  • KO/PT-BR observability: Datadog minor image frame tweaks

Developer-facing migration

  • Before:
 config.embedder = { provider, config: { model, ... } }
  • After:
config.embedding_model = { provider, config: { model, ... } }
config.vectordb = { provider: "chromadb"|"qdrant", config: { settings? | vectors_config? } }

Minimal example

from chromadb.config import Settings
tool = PDFSearchTool(
  config={
    "embedding_model": {
      "provider": "openai",
      "config": {"model": "text-embedding-3-small"}
    },
    "vectordb": {
      "provider": "chromadb",
      "config": {
        # "settings": Settings(persist_directory="/content/chroma", allow_reset=True, is_persistent=True),
      }
    }
  }
)

Notes

  • Collection name is managed by the adapter (default: "rag_tool_collection").
  • OpenAI key auto-read from env; “model” is mapped to “model_name”.

Copy link
Contributor

@lucasgomide lucasgomide left a comment

Choose a reason for hiding this comment

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

:shipit:

@tonykipkemboi tonykipkemboi merged commit 410db1f into crewAIInc:main Oct 27, 2025
42 checks passed
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