BPIManager / BPIManager-Core

BPIManager のフロントエンド開発用リポジトリ
https://bpi.poyashi.me
10 stars 7 forks source link

BPIにNaNが含まれる場合の統計画面推移タブの修正 #51

Open syota0304 opened 1 month ago

syota0304 commented 1 month ago

概要

BPIにNaNが含まれる場合に統計画面推移タブの表示に異常がある問題の修正です。

修正点

課題

総合BPIの計算箇所に修正を加えています。この修正でBPI NaNは-15として計算されます。 https://github.com/BPIManager/BPIManager-Core/pull/5#issue-1166483762 に於いて計算エラーがあったBPIの扱いが定められておりません。

NaNの発生原因が定義データ起因であれば、BPI算出対象として扱わずに総楽曲数nを-1するべきと思います。 プレーヤーの入力したデータ起因であれば、-15として扱っても良いと思います。

しかし、NaNの発生原因をハンドリングするような仕組みが無いので現実的には-15で統一せざるを得ないと思います。