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

Split `forgelm/trainer.py` into a sub-package (`_kwargs`, `_runtime`, `_finalize`, `_artifacts`) — deferred until Phase 16 S4 and S8 have landed

Đang mở
  • Có gì không ổn?
#904 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

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

Đá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
18/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
machine-learning

Hướng nghiên cứu

Start in forgelm/trainer.py and the _DEFERRED_SPLITS entry in tools/check_module_size.py, then confirm the Phase 16 S4 (#890) and S8 (#894) preconditions have landed. The work is a behaviour-neutral split into a forgelm/trainer/ sub-package, keeping ForgeTrainer and train importable from the old path, with test patch targets retargeted. Done means the module-size check passes without the deferral entry and the deferral row and CHANGELOG entry are updated.

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

Mô tả

area: trainer deferred severity: low source: roadmap wave: 4

Deferred work · F-PR29-A1-05: trainer.py · severity Low · cost ~1 day · recorded in docs/roadmap/risks-and-decisions.md:79

Summary

forgelm/trainer.py is a deferred module split: the ForgeTrainer god-object holds TRL kwarg fold-in, OOM/DeepSpeed runtime, artifact finalisation and the compliance / model-card / deployer hand-off in one file; forgelm.trainer.ForgeTrainer and forgelm.trainer.train must stay importable. The module-size guard holds it to a fatal no-growth budget, but nothing pays the debt down until someone takes the split.

Condition or budget

Phase 16 S2, S3, S4 and S8 have all landed (decision C-18: never split ahead of the behaviour steps that change the same file); S4 is #890 and S8 is #894

State today (re-checked 2026-10-09)

1519 code lines against a budget of 1519 at 08f2426 (deferred at 1432, raised five times); no forgelm/trainer/ package exists.

Related

#667 and #668 track how the module-size guard counts lines and accepts budget raises; decide them before relying on a module's budget.

Acceptance criteria

  • The module becomes a sub-package along the recorded seams, behaviour-neutral, with every public name still importable from its current path.
  • Its _DEFERRED_SPLITS entry is removed from tools/check_module_size.py and every resulting module is under the normal ceiling.
  • Test patch targets that named the old module are retargeted to the module that resolves the name at call time (the forgelm/verify.py split found five that silently moved).
  • The row is removed from the deferral table and the CHANGELOG names its ID (the table's removal contract).

Recorded in docs/roadmap/risks-and-decisions.md (read at f94595f) and re-checked against the code on 2026-10-09. The row leaves that table only when the fix is on main, the CHANGELOG names its ID and any promised guard exists.

Ngôn ngữ chính
Python
Star
9
Fork
1
Merge trung bình
17 giờ 14 phút
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 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.