mlco2/codecarbon

API client to throw error instead of None

Fermée

#820 ouverte le 13 avr. 2025

Voir sur GitHubGuide contributeur
 (6 commentaires) (0 réaction) (0 personne assignée)Python (306 forks)auto 404
enhancementgood first issue

Métriques du dépôt

Stars
 (1 885 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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

Guide contributeur

Direction de recherche
Localisez le code du client API qui renvoie None en cas d'erreur et modifiez le pour lever une exception. Ensuite, mettez à jour le emissionsTracker pour capturer cette exception.
Stack technique
python
Domaine
apibackend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PythonGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75