microsoft/InnerEye-DeepLearning

Model baseline comparison shouldn't allow duplicates

Open

#506 aberto em 23 de jun. de 2021

Ver no GitHub
 (0 comments) (0 reactions) (1 assignee)Python (519 stars) (138 forks)batch import
good first issuenot urgent

Description

The model config parameter comparison_blob_storage_paths takes a list of tuples, (name, container). The name is purely descriptive , and has no further meaning, but it is important that they are distinct otherwise multiple datasets from different containers may be merged in an unpredictable way. Essentially, it is better if this is a dict[name, container]. Add this validation to the validate function.

AB#4149

Guia do colaborador

Pilha de tecnologia
pythonazure
Domain
backendmachine learning
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
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
PythonInnerEye DeepLearning project structure
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50
Direção de pesquisa
The issue requests adding validation to the 'validate' function to ensure names in 'comparison blob storage paths' are unique. The change involves either converting the list of tuples to a dictionary or adding a duplicate check in the existing validation logic. Look for the configuration validation module in the repository, and note that there is an assignee, so coordinate with them before starting.