FluxML/Zygote.jl

Convert all adjoints to rrules

Open

#1.209 geöffnet am 20. Apr. 2022

Auf GitHub ansehen
 (3 Kommentare) (5 Reaktionen) (0 zugewiesene Personen)Julia (220 Forks)batch import
ChainRulesenhancementgood first issueup for grabs

Repository-Metriken

Stars
 (1.568 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

We have ZygoteRuleConfig for this, so there should be no major technical limitations. rrules generally have far better UX than @adjoints (as anyone who has had to read a stacktrace from Zygote can attest). The ultimate end goal would be to get rid of ZygoteRules entirely.

Contributor Guide

Research-Richtung
Untersuchen Sie jede @adjoint Funktion im Codebase und ersetzen Sie sie durch eine äquivalente rrule unter Verwendung von ZygoteRuleConfig. Stellen Sie sicher, dass alle Tests bestehen, und entfernen Sie nach und nach die Abhängigkeit von ZygoteRules.
Tech Stack
Keine
Domain
backendmachine learning
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
3-5 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Keine
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30