PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Enabling "Include in Full DPS" breaks the gem comparison window for the main skill

オープン
#5,906 コメント 5 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

bug: behaviour
主要言語
Lua
スター
5.4k
フォーク
2.4k
平均マージ
1日 12時間
マージ済み PR(30日)
26

説明

Check version
  • I'm running the latest version of Path of Building and I've verified this by checking the changelog
Check for duplicates
  • I've checked for duplicate issues by using the search function of the issue tracker
Check for support
  • I've checked that the behaviour is supposed to be supported. If it isn't please open a feature request instead (Red text is a feature request).
What is the behaviour in-game?

N/A

What is the behaviour in Path of Building?

When turning on "Include in full DPS" the "new support gem window" comparison seems to compare the main skill to the extra skill enabled.

With Orb of Storms enabled:
image

With Orb of Storms disabled:
image

How to reproduce the issue
  • Create a new character
  • Add a Main skill and one set to "Include in full DPS"
  • With the Main skill selected add the Stun support (Since Stun support shouldn't alter anything it's easy to see the bug)
  • Notice the wild comparison numbers

Can also import the attached pob and enable the "Enabling this bugs out the Gem comparison window" socket group and see the bug happening with the main skill

Character build code
https://pobb.in/YXM10r2l6M09
Screenshots

Tested also on a new POB. Arc included in full dps:
image

Arc not included:
image

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

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

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

ジェム比較ウィンドウから開始し、Stun support またはリンクされたビルドコードを使って、メインスキルと "Include in full DPS" を有効にした別のスキルで問題を再現します。ソースファイルやテストは指定されていません。選択したメインスキルの比較が、追加のスキルを完全な DPS に含めた場合でも含めない場合でも正しいままであれば完了です。

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

評価

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

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

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