mlco2/codecarbon

API client to throw error instead of None

Chiusa

#820 aperta il 13 apr 2025

Vedi su GitHubGuida contributor
 (6 commenti) (0 reazioni) (0 assegnatari)Python (306 fork)auto 404
enhancementgood first issue

Metriche repository

Star
 (1885 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

The Api client should raise_for_status() instead of None if there is an error. Like this we can call the API client and give a more meaningful error to the user.

Things to consider:

  • The API client should throw but the emissionsTracker should not break, this means that we should try-catch in the emissions tracker.
  • One should check as well if in any of the responses we are if response is None because now they will throw the error

Originally posted by @inimaz in https://github.com/mlco2/codecarbon/issues/798#issuecomment-2800035595

Guida contributor

Direzione di ricerca
Individua il codice del client API che restituisce None in caso di errore e modificalo per sollevare un'eccezione. Quindi aggiorna emissionsTracker per catturare tale eccezione.
Tech stack
python
Dominio
apibackend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75