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

Choosing-a-trainer table documents KTO rows as {prompt, response, label}, but the KTO loader requires a completion column (F-W1-DOCS-TRAIN-02)

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

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

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
75/100
Loại issue
Tài liệu
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
markdown, python
Lĩnh vực
documentation, machine-learning

Hướng nghiên cứu

Bắt đầu với dòng KTO ở dòng 51 của docs/usermanuals/en/concepts/choosing-trainer.md, sau đó kiểm tra yêu cầu cột KTO trong nhánh _validate_trainer_columns của forgelm/data.py quanh các dòng 600-607. Đổi dòng đó thành {prompt, completion, label: bool} trong trang tiếng Anh và trong bản phản chiếu tiếng Thổ Nhĩ Kỳ docs/usermanuals/tr/concepts/choosing-trainer.md ở dòng 51. Giữ nguyên dòng SFT ở dòng 49 cho #403. Hoàn thành khi cả hai ngôn ngữ khớp với bộ nạp và trang KTO.

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

Mô tả

documentation good first issue severity: medium source: review-2026-09 wave: 3

Wave 3 · group W3 Docs · highest severity Medium · 1 finding(s) · effort docs fix

Summary

The "What to look at in your data" table says {prompt, response, label: bool} rows mean KTO. The KTO loader requires completion and label and stops with a KeyError on response, so data shaped the way the page recommends does not load.

Primary finding

F-W1-DOCS-TRAIN-02 — The choosing-a-trainer table names the wrong KTO answer column

severity Medium · confidence high · checked against the loader

What happens. docs/usermanuals/en/concepts/choosing-trainer.md:51 and the Turkish mirror at the same line. _validate_trainer_columns raises KeyError: KTO trainer requires 'completion' and 'label' (boolean) columns … (forgelm/data.py:600-607). The row above it (:49, {prompt, completion} → SFT) is the SFT-format defect tracked in #403; that issue now lists this page too.

Where.

Evidence (re-verified on 2026-10-09).

  • The KTO branch of _validate_trainer_columns names only completion and label; both table rows are unchanged on development.

Recommended fix. Change the KTO row to {prompt, completion, label: bool} in both languages, matching the KTO page and the loader. Settle the SFT row (:49) with whatever #403 decides for the SFT format.

Related. #403 — the SFT row of the same table; fix both rows together if the PRs overlap.

Acceptance criteria

  • Every data shape in the table loads with the trainer it recommends.
  • Documentation listed above is corrected in English and in its Turkish mirror in the same PR.
  • The finding is resolved or explicitly closed with a reason.

Found by the September 2026 independent review of ForgeLM against commit cab2947 and re-verified against development at f94595f on 2026-10-09. Code links point at cab2947; line numbers may have moved since. Finding IDs (F-W1-…) are stable references for this backlog.

Ngôn ngữ chính
Python
Star
9
Fork
1
Merge trung bình
8 giờ 10 phút
Pull request đã merge (30 ngày)
5

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 HodeTech/ForgeLM

Tất cả issue của HodeTech/ForgeLM

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.