Skip to content

fix: post-merge fixes for Contextual Retrieval (Part B crash, LOG_LEVEL, frontend 0%)

Jérôme Revillard requested to merge feat/contextual-retrieval into main

Post-merge fixes for Contextual Retrieval (!199 (merged))

4 commits pushed to feat/contextual-retrieval after !199 (merged) was merged:

  1. 37bacd192 chore(config): wire Part B (hybrid BM25+RRF) vars — Part B added vars to config.py + env template but missed docker-compose (retriever service) + env.j2. Without this, BM25 hybrid can't be enabled via deploy.

  2. cdfdd6a23 fix(retriever): use views() not has_view() — Part B's _ensure_bm25_view called self.db.has_view() which doesn't exist in python-arango 8.3.3 → AttributeError → BM25 view never created → 404 → silent vector-only fallback. Fixed to use views() membership.

  3. 86f4913d0 fix(chatqna): respect LOG_LEVEL + CONFDEBUG — the OPEA CustomLogger never set the level from LOG_LEVEL (DEBUG never worked). Patches uvicorn's LOGGING_CONFIG dict (root + handler + app logger) so DEBUG fires. Default INFO → prod-safe. Strengthened the CONF:N instruction (forceful + 1-shot example).

  4. 6af655e87 fix(frontend): display confidence badge even at 0%if (confidence_score) treats 0 as falsy → badge hidden. Changed to != null. Mobile already correct (Dart).

All verified: 572 tests pass, ruff clean, config-validator 25/25.

Merge request reports

Loading