Lightning-AI/pytorch-lightning

Refactor LearningRateMonitor

Open

#17 613 ouverte le 11 mai 2023

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Python (3 233 forks)batch import
callback: lr monitorfeaturehelp wantedrefactor

Métriques du dépôt

Stars
 (26 687 stars)
Métriques de merge PR
 (Merge moyen 9j 15h) (3 PRs mergées en 30 j)

Description

Outline & Motivation

  • self.last_momentum_values does not seem needed, but I may be overlooking something.
  • Instead of returning the name with -momentum suffix when logging betas, just log it with -beta suffix.
  • If logging betas, log all of them. The name could therefore be -beta0, -beta1 in the case of Adam.
  • Change the current logging names, e.g., lr-Adam to optimizer-Adam-lr and lr-Adam-momentum to optimizer-Adam-momentum (it doesn't have momentum this is just for demonstration, it would actually be optimizer-Adam-beta0, optimizer-Adam-beta1).
  • remove the log_momentum argument as it can be inferred automatically.

Pitch

Happy to open a PR with these changes.

Additional context

No response

cc @borda @justusschock @awaelchli

Guide contributeur

Direction de recherche
Examinez l'implémentation actuelle de LearningRateMonitor dans la base de code (probablement dans `pytorch lightning/callbacks/lr monitor.py`). Identifiez où `self.last momentum values` est utilisé et évaluez s'il est redondant. Planifiez le renommage des clés de journalisation comme décrit. Envisagez également de supprimer l'argument `log momentum` en détectant automatiquement la présence du momentum. Passez en revue les tests existants pour le rappel afin de garantir que les modifications sont testées.
Stack technique
python
Domaine
machine learningaibackend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PythonPyTorch LightningGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70