EnergySystemsModellingLab / EnergySystemsModellingLab/MUSE_OS

Change interpolation method for capacity data

オープン
#656 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Python
スター
29
フォーク
12
PR マージ指標
30日以内にマージされた PR はありません

説明

The `interpolate_capacity` function is currently using linear interpolation. However, as pointed out [here](https://github.com/EnergySystemsModellingLab/MUSE_OS/pull/642#discussion_r1944376005), linear interpolation doesn't make much sense because capacity profiles are step-wise. Forward filling (i.e. taking the previous specified value) would make more sense.

We can simply change `interpolate_capacity` to return `data.reindex(year=year, method="ffill")`

I've given this a try, but some of the results changed and I'd like to understand why before implementing this change

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

interpolate_capacity 関数から始め、その呼び出し元を調べて、容量データがどのようにインデックス付けされ、なぜ前方補完によって結果が変わるのかを理解します。影響を受けるすべての容量プロファイルについて、現在の線形補間と data.reindex(year=year, method="ffill") を比較します。変更する前に、補間の動作を理解し、意図した結果を確認できれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
python
領域
data-engineering
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。