mlco2/codecarbon

API client to throw error instead of None

クローズ

#820 opened on 2025/04/13

 (6 件のコメント) (0 件のリアクション) (0 人の担当者)Python (306 件のフォーク)auto 404
enhancementgood first issue

Repository metrics

Stars
 (1,885 個のスター)
PR merge metrics
 (PR metrics pending)

説明

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

コントリビューターガイド