get_cloud_geo_region() returns NaN instead of the city for GCP regions with no state
メンテナーはふだん 2 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 86/100
調査の方向性
codecarbon/core/emissions.py の get_cloud_geo_region() から始め、data/cloud/impact.csv から読み込まれる state の値がどのように処理されるかを調べてください。GCP リージョン asia-east1 で再現し、その後、state のないリージョンのテストを追加してください。都市が返され、そのリージョンが排出量の出力で NaN を生成しなくなれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Problem
Emissions.get_cloud_geo_region() (codecarbon/core/emissions.py) falls back to city only when state is None:
state = selected["state"].item()
if state is not None:
return state
impact.csv is read with pandas, so an empty state cell is NaN, not None. The check passes and NaN is returned. 30 of the 40 GCP regions in data/cloud/impact.csv have no state (e.g. asia-east1, africa-south1), so the city is never used.
Reproduction
from codecarbon import OfflineEmissionsTracker
tracker = OfflineEmissionsTracker(cloud_provider="gcp", cloud_region="asia-east1")
tracker.start()
data = tracker._prepare_emissions_data()
tracker.stop()
print(repr(data.region)) # nan, expected 'Changhua County'
Impact
regionis empty inemissions.csvHTTPOutputfails:requestsraisesInvalidJSONError: Out of range float values are not JSON compliant: nanLoggerOutputlogs invalid JSON ("region": NaN)
Proposed fix
Check for a real string instead of None, e.g. if isinstance(state, str):, and add a test with a region that has no state. Happy to open a PR.
Proposed fix
Check for a real string instead of None, e.g. if isinstance(state, str):, and add a test with a region that has no state. Happy to open a PR.
- 主要言語
- Python
- スター
- 1.9k
- フォーク
- 341
- 平均マージ
- 3日 51分
- マージ済み PR(30日)
- 16
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
mlco2/codecarbon のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
mlco2/codecarbon#1460 ·
メンテナーはふだん 2 日以内に返信
-
count_cpus() and RAM crash with FileNotFoundError when SLURM_JOB_ID is set but scontrol is missingオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
mlco2/codecarbon#1455 ·
メンテナーはふだん 2 日以内に返信
-
CLI: get_config() returns None without [codecarbon] section; get_api_endpoint() appends api_endpoint under wrong section対応中かも @sohammishra864-wq が 27 日前に担当しました。 オープン
難易度 2/5 半日 初心者へのやさしさ 82/100
mlco2/codecarbon#1432 ·
メンテナーはふだん 2 日以内に返信
-
Update the screen shot of the dashboard対応中かも @aarthis81 が 22 日前に担当しました。 オープンdocumentation good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
mlco2/codecarbon#1423 · コメント 5 件 ·
メンテナーはふだん 2 日以内に返信
-
Update the bibtex citation対応中かも @davidberenstein1957 が 75 日前に担当しました。 オープンdocumentation good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
mlco2/codecarbon#1422 · コメント 5 件 ·
メンテナーはふだん 2 日以内に返信
mlco2/codecarbon の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
sylvainmetayer/homelab#851 ·
メンテナーはふだん 1 日以内に返信
-
needs triage
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
kubeflow/docs-agent#279 ·
メンテナーはふだん 3 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
Qiskit/qiskit-ibm-runtime#3523 ·
メンテナーはふだん 1 日以内に返信