mlco2/codecarbon

API client to throw error instead of None

Geschlossen

#820 geöffnet am 13.04.2025

Auf GitHub ansehenContributor Guide
 (6 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (306 Forks)auto 404
enhancementgood first issue

Repository-Metriken

Stars
 (1.885 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Finden Sie den API Client Code, der bei einem Fehler None zurückgibt, und ändern Sie ihn so, dass eine Ausnahme ausgelöst wird. Aktualisieren Sie dann den emissionsTracker, um diese Ausnahme abzufangen.
Tech Stack
python
Domain
apibackend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
PythonGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75