PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Allow Toggle to shut off 'Have you attacked recently' and 'Have you cast a spell recently' when primary skill is tagged Attack/Spell

Đang mở
#5,153 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: behaviour
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 issues by using the search function of the issue tracker
Is your feature request related to a problem?

The current implementation of 'Have you attacked Recently' and 'Have you cast a spell recently' checkboxes assumes true if the primary skill gem has the Attack or Spell tag respectively.

I'm running a holy relic guardian. My main skill is 'Summon Holy Relic'. This is the skill I normally want to see the DPS/stats from in PoB. This gem has the 'Spell' tag, so PoB is using this to assert that I've cast a spell recently.

I've taken the ascendency node Bastion of Hope, which states 'If you've cast a spell recently, you and nearby allies have 10% chance to block spell damage'.

However, holy relics are semi-permanent.... the only time I cast them is when they die, which is rare. I use cyclone to trigger the holy relics, so I'm channeling, but rarely actually casting a spell. So most of the time, 'Have you cast a spell recently' is false, I have not. In-game I will not usually have this extra spell block chance.

You can switch the skill to a different one in PoB (i.e. an attack skill, cyclone) to see the right spell block values, but that's really a workaround.

Also, switching to cyclone adds regular block chance, since bastion of hope also states 'If you've attacked recently, you and nearby allies have 10% chance to block attack damage'. But this is also often false. This is only triggered when you 'attack', which appears to only be true with cyclone when you initially engage the skill. As you channel cyclone, this buff falls off, regardless of whether you're hitting enemies. (tested in-game while observing the tooltip window).

If you're mostly spinning-to-win and just holding cyclone on, you won't be considered to have attacked recently most of the time either.

Describe the solution you'd like

It would be nice if we could forcibly disable the 'Have you attacked recently' and 'Have you cast a spell recently' checkboxes when they're assumed to be enabled due to the gem tags. The current implementation is misleading for builds that use minion (spell) skills and channeling attack skills like cyclone.

Describe alternatives you've considered

Switching the skill to a non-attack or non-spell skill shows the correct values, but it means I have to switch skills each time I mess with the tree/gear setups in order to see accurate block/spell block chances.

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

Issue không nêu tên tệp, bài kiểm thử hoặc điểm truy cập nào. Hãy bắt đầu bằng cách tìm các control 'Have you attacked recently' và 'Have you cast a spell recently', cùng với logic suy ra giá trị của chúng từ các tag kỹ năng chính. Được xem là hoàn thành khi người dùng có thể tắt rõ ràng các trạng thái được suy ra đó mà vẫn duy trì các phép tính chính xác về block và spell-block.

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