[Bug]: Four evaluation scripts unpack init_memmachine_params's five return values into three names

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#1,911 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 3 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
1/5
Thời gian dự kiến
Dưới một giờ
Mức phù hợp với người mới
88/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
tooling

Hướng nghiên cứu

init_memmachine_params trong evaluation/utils/agent_utils.py hiện trả về năm giá trị, nhưng bốn nơi gọi vẫn unpack ba giá trị: evaluation/episodic_memory/locomo_ingest.py, locomo_search.py, locomo_delete.py và longmemeval_test.py (longmemeval_delete). Hãy so sánh chúng với một nơi gọi đã unpack năm giá trị, chẳng hạn các nơi gọi trong evaluation/retrieval_agent/. Hoàn thành khi mỗi dòng trong bốn dòng đó unpack đủ năm giá trị trả về và một trong các script LoCoMo chạy qua được dòng đó.

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

Mô tả

Describe the bug

Paths and line numbers are at 377510d43 (main).

init_memmachine_params in evaluation/utils/agent_utils.py returns five values: (memory, answer_model, query_agent, agent_model_id, answer_model_id). Four call sites unpack three names, so each raises ValueError: too many values to unpack (expected 3) before the script does any work:

Call site Function
evaluation/episodic_memory/locomo_ingest.py:61 LoCoMo ingest
evaluation/episodic_memory/locomo_search.py:177 LoCoMo search
evaluation/episodic_memory/locomo_delete.py:52 LoCoMo delete
evaluation/retrieval_agent/longmemeval_test.py:260 longmemeval_delete

The other call sites, in evaluation/retrieval_agent/ and evaluation/retrieval_agent/beam/, unpack all five.

Steps to reproduce

Run any of the three evaluation/episodic_memory/locomo_*.py scripts, or LongMemEval's delete run type, with a valid configuration. Each fails at the line in the table.

Expected behavior

Each call site unpacks the five values the function returns.

Additional context

#1376 changed the return from three values to five and updated the call sites it touched. It did not touch the evaluation/episodic_memory/ LoCoMo scripts, and it missed the call in longmemeval_delete. CI does not run or type-check evaluation/.

🤖 Written by Claude Code (Claude Opus 5.5) on behalf of @edwinyyyu.

Ngôn ngữ chính
Python
Star
3.1k
Fork
217
Merge trung bình
4 ngày 23 giờ
Pull request đã merge (30 ngày)
17

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

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 MemMachine/MemMachine

Tất cả issue của MemMachine/MemMachine

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.