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

Ground truth: decide on gradient validation (currently fully unimplemented)

Đang mở
  • Có gì không ổn?
#986 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ó
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
25/100
Loại issue
Tính năng
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
kotlin, python
Lĩnh vực
machine-learning, testing

Hướng nghiên cứu

Start by reading gt/pytorch/io/writer.py, GroundTruthLoader.kt, GroundTruthValidator.validateGradientsInternal, CONTRACT.md, and TESTING.md to understand what is implemented and documented. Decide whether gradient validation should be implemented on both sides or explicitly de-scoped; done means the chosen path is made consistent across code and documentation, including removing misleading unused API surface if de-scoped.

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

Mô tả

assessment question sub-issue

Sub-issue of #984.

Despite the "gradienttracer" name, no gradient is ever actually checked today.
ToleranceConfig.GRADIENT, GroundTruthTestCase.expectedGradients, and grad_*-tensor
parsing all exist on the Kotlin side (GroundTruthLoader.kt,
GroundTruthValidator.validateGradientsInternal) — but store_experiment_as_gguf
(gradienttracer's gt/pytorch/io/writer.py) never calls .backward() or writes any
grad_* tensor. validateGradientsInternal is explicitly a placeholder (// TODO: Implement actual gradient computation via tape) that always reports failure if ever
invoked with real data.

Decision needed

  • Implement it: add .backward() + grad_* tensor serialization to
    store_experiment_as_gguf (every @Executable input already has requires_grad=True
    set, so the gradient is available, just not captured), and implement SKaiNET-side
    backward-pass execution in GroundTruthValidator.validateGradientsInternal to actually
    compare against it. This is real, nontrivial work on both sides — SKaiNET's own
    autodiff/backward execution path needs to be driveable from
    skainet-test-groundtruth, which it currently isn't.
  • Explicitly de-scope it: if gradient validation isn't a near-term priority, say so
    in CONTRACT.md and TESTING.md (which currently reads as if this already works),
    and consider whether the dead expectedGradients/ToleranceConfig.GRADIENT/
    validateGradients parameter surface should be removed rather than left as
    looks-implemented-but-isn't.

Either answer is fine — the point of this issue is picking one, since right now the code
reads as "supported" when it silently isn't.

Ngôn ngữ chính
Kotlin
Star
52
Fork
15
Merge trung bình
1 ngày 15 giờ
Pull request đã merge (30 ngày)
36

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 SKaiNET-developers/SKaiNET

Tất cả issue của SKaiNET-developers/SKaiNET

Issue tương tự

Thêm issue về Kotlin

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.