PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Ramp Time in Side Bar

Đang mở
#306 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ả

I'd like ramp time to be included whenever the calculation mode combines hit dps with poison/impale dps. It's important context for the final dps number that players can miss when talking about X bajillion viper strike dps.

If the box is getting too crowded then maybe replace the "total damage per poison" stat, which doesn't seem very relevant when dps is concerned.

ramp time for poison dps = poison duration
ramp time for impale dps = max number of impales / attacks per second

Originally posted by @Sectoidfodder in https://github.com/PathOfBuildingCommunity/PathOfBuilding/pull/158#issuecomment-594084168

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

Không có tệp hoặc test nào được nêu tên. Hãy bắt đầu bằng cách tìm phần hiển thị chế độ tính toán trong thanh bên và các chỉ số poison, impale và total-damage-per-poison hiện có; so sánh các công thức ramp time được yêu cầu với các phép tính hiện tại. Công việc được xem là hoàn tất khi các chế độ DPS của hit kết hợp với poison/impale hiển thị ramp time mà không làm thanh bên bị quá tải.

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ó
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

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.