PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
DPS preview when swapping out lynchpin gem shows output for next default skill, sometimes misleadingly
まだ誰も着手していません。
- 主要言語
- Lua
- スター
- 5.4k
- フォーク
- 2.4k
- 平均マージ
- 1日 12時間
- マージ済み PR(30日)
- 26
説明
- What happened?
Messing with supports for a minion, I had their Infernal Legion DPS selected for Main Skill. When I then moved to the dropdown for picking a different support in the Infernal Legion slot, the preview tooltip claimed that changing to Elemental Focus would give me an unrealistic +1M DPS boost.
If I select the gem previewed, it changes my Main Skill to Minion Instability, with the resulting DPS matching the "+DPS inc. Poison" part of the strange preview. I'm not sure where the +1M Total DPS comes from.
- What were you trying to do?
No clear use case here--you typically don't want to be swapping out the gem that gives you your "Main Skill" DPS anyway. It was an accidental encounter while exploring gem options. But in a less obviously-wrong case it could mislead someone about their DPS prospects!
- What steps will reproduce the bug?
- Create a new build
- Add a minion skill and Infernal Legion support
- Go to the dropdown from Infernal Legion and hover over a different support. Note the possibly strange preview numbers
- Select the new gem. The Main Skill changes, with its numbers resembling the preview.
- Does it reproduce every time? If not, why so?
Yes, although the specific numbers vary wildly depending on the rest of the build's setup.
- Additional information:

Suggested fix would be to have any such preview say that it's removing all the DPS, making clear that you're effectively deleting that source of damage.
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
新しいビルドを作成し、Infernal Legion を付けた minion スキルを追加して、そのドロップダウンで別の support にカーソルを合わせて問題を再現します。プレビューとジェム選択のエントリーポイントを追跡して Main Skill が変わる理由を確認し、その後、プレビューが現在の DPS ソースの削除を明確に反映し、誤解を招く合計値を報告しなくなったことを検証します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- lua
- 領域
- desktop
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 45/100