SheffieldML/GPy

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

Open

#922 aperta il 27 mag 2021

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Python (579 fork)batch import
good first issue

Metriche repository

Star
 (1927 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Rivedi la discussione PR collegata all'indirizzo https://github.com/SheffieldML/GPy/pull/917#discussion r639520080 per capire quali asserzioni specifiche dovrebbero essere sostituite. Quindi cerca nel codice sorgente di GPy tutte le istanze di `assert` che convalidano l'input dell'utente (anziché asserzioni interne) e sostituiscile ciascuna con un appropriato `raise ValueError(...)`. I file comuni includono moduli in `GPy/core/` o altro codice di gestione dell'input.
Tech stack
python
Dominio
developer experiencemachine learning
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Python basicsUnderstanding of exceptions
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90