PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Sorting uniques yields incorrectly sorted items when main skill is minion based

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

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

bug
主要言語
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

image

Check for duplicates
  • I've checked for duplicate issues by using the search function of the issue tracker
What is the expected behaviour/value?

I understand the sorting might be tricky because some sorting options need to apply to minions, such as DPS or life when a minion skill is selected.

Except some sorting options seem to player-only (strength, dexterity, intelligence, effective hit pool, and so on). So for these, I would expect to have items being sorted based on the character itself rather than the skill selected.

What is the actual behaviour/value?

When sorting unique items, many of the sorting options yields the wrong results.
I identified two cases where sorting results are either alphabetically sorted (case 1) or seemingly random (case 2)

The root cause of the issue seems to be linked with having a Main Skill configured on a minion skill (in my case Absolution).
Selecting a non-minion skill causes the sorting option to yield an adequately sorted list.

Case 1 - Sorting options yielding alphabetically sorted list

Examples (with 1st result highlighted)

  • Sort by Mana:
    image

  • Sort by Strength:
    image

  • Sort by Dexterity:
    image

Case 2 - Sorting options yielding list in seemingly random order

  • Sort by Effective Hit Pool:

    • 1st item: image
    • 2nd item: image
    • 10th item: image
How to reproduce the issue
  1. Add any minion skill as main skill, for example: Absolution
  2. Sort unique items by metrics only affecting the player, for example: Strength, Dexterity, Intelligence or Effective Health Pool
  3. Then switch the main skill to a non-minion skill => results are different from step 2.
Character build code
https://pobb.in/2pE4Szyp5TuB
Screenshots

No response

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

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

はじめの一歩

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

調査の方向性

issue ではソースファイルもテストも特定されていません。リンクされたキャラクタービルドを読み込み、Absolution などの minion スキルを選択して、Strength、Dexterity、Mana、または Effective Hit Pool で unique アイテムをソートする状況を再現し、その後 minion ではないメインスキルと比較します。選択したスキルに関係なくプレイヤーのみのメトリクスが一貫してソートされ、報告されたケースに対する回帰テストのカバレッジがあることを完了条件とします。

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

評価

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

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

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