Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

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

Ouverte
  • Quelque chose ne va pas ?
#986 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
5/5
Temps estimé
Plus d'une semaine
Accessibilité débutants
25/100
Type d'issue
Fonctionnalité
Clarté
Plutôt claire
Activité
Calme
Stack technique
kotlin, python
Domaine
machine-learning, testing

Piste de recherche

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.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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.

Langage dominant
Kotlin
Étoiles
52
Forks
15
Merge moyen
1 j 2 h
PR mergées (30 j)
63

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de SKaiNET-developers/SKaiNET

Toutes les issues de SKaiNET-developers/SKaiNET

Issues similaires

Plus d'issues Kotlin

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.