`--provider openai-codex` is offered by commands whose answerer or generator refuses it

Đã đóng
  • Có gì không ổn?
#404 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

Một pull request liên quan đã được merge.

  • #428 của @vlad-perevezentsev — đã merge

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
56/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
ai, cli

Hướng nghiên cứu

Start by reading the definitions of ANSWER_PROVIDERS and CALLER_PROVIDERS in dokimi.llm, then trace how the listed commands validate --provider and route it to the LLM. The issue offers two paths: accept openai-codex or remove it from command choices; accepting it requires confirming the model works through chat completions. Done means the commands no longer offer a provider they reject, with tests covering the selected behavior.

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

Mô tả

Found in the review of #384 (https://github.com/intel/dokimi/pull/384#discussion_r4473774384). Pre-existing: main behaves the same.

--provider on these commands takes llm.ANSWER_PROVIDERS, which includes openai-codex. The class each command builds checks llm.CALLER_PROVIDERS, which does not. So the run stops with ValueError: Unsupported provider: 'openai-codex':

Command Refused by
answers generate Answerer
questions generate QuestionGenerator
personas discover PersonaGenerator
evaluate, benchmark run, benchmark batch EvaluationPipeline → PersonaGenerator, QuestionGenerator, Answerer

dokimi.llm already routes the provider (openai/<model>, key OPENAI_API_KEY). The callers' tuples have left it out since before #384; that PR only merged the four copies into one constant.

Two ways to fix it, one PR:

  1. Accept it: add openai-codex to CALLER_PROVIDERS, with one test per command above that it reaches dokimi.llm with the openai/ route.
  2. Stop offering it: build these commands' --provider choices from CALLER_PROVIDERS (argparse then exits 2 before the run starts).

Option 1 needs someone to confirm that the models openai-codex names answer on that route through chat completions; if that is not known, option 2 is the safe one.

Ngôn ngữ chính
Python
Star
11
Fork
9
Merge trung bình
1 ngày 9 giờ
Pull request đã merge (30 ngày)
426

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 intel/dokimi

Tất cả issue của intel/dokimi

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.