SheffieldML/GPy

There are `assert`s where we need `ValueError`s

Open

#922 aberto em 27 de mai. de 2021

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Python (579 forks)batch import
good first issue

Métricas do repositório

Stars
 (1.927 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

We should not use assert to address bad user input, but there are such uses of assert in the GPy codebase. Instead, we should use ValueError. For more details, see https://github.com/SheffieldML/GPy/pull/917#discussion_r639520080

Guia do colaborador

Direção de pesquisa
Revise a discussão do PR vinculada em https://github.com/SheffieldML/GPy/pull/917#discussion r639520080 para entender quais asserções específicas devem ser substituídas. Em seguida, pesquise na base de código do GPy por todas as instâncias de `assert` que validam a entrada do usuário (em oposição a asserções internas) e substitua cada uma por um `raise ValueError(...)` apropriado. Arquivos comuns incluem módulos em `GPy/core/` ou outro código de manipulação de entrada.
Pilha de tecnologia
python
Domain
developer experiencemachine learning
Tipo Issue
Refatoração
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.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Python basicsUnderstanding of exceptions
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
90