microsoft/InnerEye-DeepLearning

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

Open

#570 aberto em 30 de set. de 2021

Ver no GitHub
 (0 comments) (0 reactions) (1 assignee)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

Guia do colaborador

Pilha de tecnologia
python
Domain
backend
Tipo Issue
refactor
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Pythonfamiliarity with the repository
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40
Direção de pesquisa
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.