Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

radius is documented and validated but has no effect on any search lane

Đang mở
  • Có gì không ổn?
#394 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@knqiufan đang làm issue này rồi.

Từ ngày 16/9/2026.

  • #451 của @knqiufan — đang mở

Đánh giá

Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức phù hợp với người mới
15/100
Loại issue
Lỗi
Độ rõ ràng
Cần làm rõ
Mức độ hoạt động
Đình trệ
Công nghệ
python
Lĩnh vực
backend, search

Hướng nghiên cứu

Start with memory/search/dto.py:84-88 and the radius handling in manager.py:120, :420, :477, :551, and :937-941; then compare the ranking behavior in everalgo/rank/hybrid.py:86-91 and fusion.py:41. Before implementing anything, resolve which radius semantics to use and whether to remove the inert RankInput.radius arguments. PR #451 is open, so check its relationship to this issue first.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

bug

SearchRequest.radius is declared, range-validated to [0.0, 1.0], and documented at memory/search/dto.py:84-88 as gating "raw cosine at recall time". It has no effect anywhere.

  • HYBRID: manager.py passes radius=_effective_radius(req) into RankInput at three call sites (~:420, :477, :551). grep -rn radius .venv/.../everalgo/ returns exactly two hits, both in types/rank.py — the field declaration and its docstring. everalgo never reads it.
  • AGENTIC: never passed at all.

PR #393 briefly wired it into the agentic path as ahybrid_retrieve(min_score=radius) and had to revert: min_score is applied after RRF fusion (everalgo/rank/hybrid.py:86-91), where rrf replaces each score with Σ 1/(k+rank) (max ≈ 0.0328, fusion.py:41). Since top_k defaults to -1 and _effective_radius then returns _DEFAULT_UNLIMITED_RADIUS = 0.5 (manager.py:120, :937-941), every candidate was filtered out — default agentic agent search returned []. Measured: {None: 2 hits, 0.5: 0 hits, 0.03: 2 hits}.

Needs a decision before any implementation: what should radius mean? Candidates — (a) a pre-fusion cosine floor applied inside the dense recall route, which is what the docstring describes; (b) a post-fusion floor on the calibrated LR score, which is what min_score already does; (c) remove it and keep min_score. Whichever is chosen, apply it consistently across lanes and delete the three inert RankInput.radius args.

Found while verifying #393.

Ngôn ngữ chính
Python
Star
13.3k
Fork
926
Merge trung bình
17 giờ 8 phút
Pull request đã merge (30 ngày)
32

Chuẩn bị môi trường

Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của EverMind-AI/EverOS

Tất cả issue của EverMind-AI/EverOS

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.