PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Power Report doesn't seem to consider Masteries.

Đang mở
#7,971 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.

enhancement
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 for duplicates
  • I've checked for duplicate open and closed issues by using the search function of the issue tracker
What platform are you running Path of Building on?

Windows

Is your feature request related to a problem?

When using Power Report, it only checks for Notables and small nodes, not Masteries.

(I also think that Power Report is broken tbh, I often get cases where the Full DPS displayed in the list for a certain path do not match the reality of actually allocating those points at all : https://i.imgur.com/mpqgYao.png)

Describe the solution you'd like

Make it so Power Report also checks for Masteries.

Describe alternatives you've considered

No response

Additional context

No response

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

Bắt đầu bằng cách xác định entry point của Power Report và cách xử lý hiện có đối với Notables và các node nhỏ. Theo dõi cách Masteries được biểu diễn và đánh giá, sau đó xác minh rằng Power Report bao gồm chúng và DPS được hiển thị khớp với việc phân bổ path được 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
Tính năng
Độ 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.