Skip to content

Conversation

@enitrat
Copy link
Collaborator

@enitrat enitrat commented Oct 4, 2025

Changes Made:

  1. pgvector_rm.py:76 - Removed openai_client parameter entirely and made embedding_func optional, defaulting to dspy.settings.embedder
  2. server/app.py:187 - Configure embedder once at startup in dspy.configure() alongside LM and adapter
  3. server/app.py:649 - Removed embedder creation in lifespan(), now uses context embedder
  4. document_retriever.py:549,619 - Removed hardcoded embedder instances, now relies on dspy.settings.embedder
  5. All 3 optimizer files - Updated to configure embedder in dspy.configure() instead of passing directly to SourceFilteredPgVectorRM

@enitrat enitrat merged commit ba1431f into main Oct 4, 2025
3 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