microsoft/InnerEye-DeepLearning

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

Open

Aperta il 30 set 2021

Vedi su GitHub
 (0 commenti) (0 reazioni) (1 assegnatario)Python (519 star) (138 fork)batch import
good first issuenot urgent

Descrizione

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

Guida contributor

Tech stack
python
Dominio
backend
Tipo issue
refactor
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 hours
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
stale
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
Pythonfamiliarity with the repository
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40
Direzione di ricerca
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.