microsoft/InnerEye-DeepLearning

We have a field "recovery_start_epoch" in deep_learning_config that can probably be retired

Open

#570 ouverte le 30 sept. 2021

Voir sur GitHub
 (0 commentaires) (0 réactions) (1 assigné)Python (519 stars) (138 forks)batch import
good first issuenot urgent

Description

recovery_start_epoch is not settable. We use it to determine the list of training epochs - but even that could be dead code. Verify and clean up

AB#4535

Guide contributeur

Stack technique
python
Domaine
backend
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Pythonfamiliarity with the repository
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40
Direction de recherche
Look in the deep learning config file (likely in InnerEye/ML/configs) for the 'recovery start epoch' field. Search the codebase for all references to this field. If it is unused, remove the field and any associated logic. Ensure no tests rely on it.