PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
Sorting uniques yields incorrectly sorted items when main skill is minion based
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Lua
- Star
- 5.4k
- Fork
- 2.4k
- Merge trung bình
- 1 ngày 12 giờ
- Pull request đã merge (30 ngày)
- 26
Mô tả
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
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:

-
Sort by Strength:

-
Sort by Dexterity:

Case 2 - Sorting options yielding list in seemingly random order
-
Sort by Effective Hit Pool:- 1st item:

- 2nd item:

- 10th item:

- 1st item:
How to reproduce the issue
- Add any minion skill as main skill, for example:
Absolution - Sort unique items by metrics only affecting the player, for example:
Strength,Dexterity,IntelligenceorEffective Health Pool - 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
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Issue không xác định tệp mã nguồn hoặc bài kiểm thử nào. Hãy tải character build được liên kết, chọn một kỹ năng minion như Absolution và tái hiện việc sắp xếp các vật phẩm unique theo Strength, Dexterity, Mana hoặc Effective Hit Pool, sau đó so sánh với một kỹ năng chính không phải minion. Được xem là hoàn tất khi các chỉ số chỉ dành cho người chơi được sắp xếp nhất quán bất kể kỹ năng được chọn, với độ bao phủ hồi quy cho các trường hợp đã báo cáo.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- lua
- Lĩnh vực
- desktop
- Loại issue
- Lỗi
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 35/100