PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Duplicate skill gems in full DPS causes incorrect DPS calculations

Đang mở
#4,682 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

bug
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.

  1. Add a bow to an empty build.
  2. Go to skills tab.
  3. Add 2 instances of Toxic Rain, one without links and the other with ballista totem. Enable "Include in FUll DPS" for both
  4. 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.

  1. Go to skills tab
  2. Select Toxic Rain (6)
  3. Attempt to change an enabled support to a new gem.

Steps to temporarily fix the bug from the PoB provided.

  1. Go to skills tab.
  2. Select TR Totem (5).
  3. Disable "Include in Full DPS."
  4. Switch to Toxic Rain (6).
  5. Attempt to change an enabled support gem to a new gem.
Character build code
https://pobb.in/hPql3eZ74-8F
Screenshots

Expected behavior
Current behavior

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. 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

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.