Skip to content

Refacto retriever - #114

Merged
Ahmath-Gadji merged 3 commits into
devfrom
refacto-retriever
Oct 22, 2025
Merged

Refacto retriever#114
Ahmath-Gadji merged 3 commits into
devfrom
refacto-retriever

Conversation

@Ahmath-Gadji

Copy link
Copy Markdown
Collaborator

Changes

  • Removed logger arguments and centralized logging.
  • Cleaned up retriever classes by removing unused parameters like criteria.
  • Updated retriever setup to take LLMs explicitly.
  • Added RETRIEVER_TYPE env variable for flexible configuration.
  • Unified LLM config handling across retrievers.
  • Fixed RagPipeline to use the correct LLM for contextualization.
  • Removed unused imports and a noisy warning in app_front.py.
  • Updated all retriever and pipeline calls to match new constructor signatures.

Result

Simpler, cleaner, and more consistent retriever and pipeline setup.

@Ahmath-Gadji
Ahmath-Gadji merged commit ddb9284 into dev Oct 22, 2025
1 check failed
@Ahmath-Gadji
Ahmath-Gadji deleted the refacto-retriever branch October 22, 2025 07:20
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.

1 participant