PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
Duplicate skill gems in full DPS causes incorrect DPS calculations
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?
DPS Calculations should accurately show more/less changes to the skill.
What is the actual behaviour/value?
DPS appears to inconsistently either show the final calc values, or an impossibly large more damage multiplier.
How to reproduce the issue
If you wish to observe from nothing. The properties here appear to be slightly different. Ballista totem needs to be present as a link for a variant of the bug to appear.
- Add a bow to an empty build.
- Go to skills tab.
- Add 2 instances of Toxic Rain, one without links and the other with ballista totem. Enable "Include in FUll DPS" for both
- Start to add support gems to the first Toxic Rain.
In the PoB provided below, ballista totem does not appear to affect the outcome of the bug.
- Go to skills tab
- Select Toxic Rain (6)
- Attempt to change an enabled support to a new gem.
Steps to temporarily fix the bug from the PoB provided.
- Go to skills tab.
- Select TR Totem (5).
- Disable "Include in Full DPS."
- Switch to Toxic Rain (6).
- Attempt to change an enabled support gem to a new gem.
Character build code
https://pobb.in/hPql3eZ74-8F
Screenshots


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
Nhập mã build được cung cấp từ pobb.in, mở tab Skills và tái hiện vấn đề bằng cách bật hai instance Toxic Rain trong Full DPS rồi thay đổi một support gem. So sánh các thay đổi sát thương more/less được hiển thị với hành vi mong đợi trong các ảnh chụp màn hình; hoàn thành khi các skill gem trùng lặp không còn tạo ra các hệ số nhân DPS không nhất quán hoặc lớn đến mức bất khả thi.
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
- game-dev
- Loại issue
- Lỗi
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 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