PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Feature Request - Display order/additional line for (CoC)Triggered spells

オープン
#3,863 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

enhancement
主要言語
Lua
スター
5.4k
フォーク
2.4k
平均マージ
1日 12時間
マージ済み PR(30日)
26

説明

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?

When I try to set up an optimal Cast on Crit setup useful information is on both on the triggering skill and the triggered skill making it a tiny bit annoying to plan around with constant tooltip swapping.

For example let's say you have a Cyclone-CoC-IceNova link. If you want to optimize you gonna have to swap back and forth Cyclone and Ice Nova details under "Main skill" menu on the left when trying to plan around getting the most optimal trigger rate, Cyclone's attack rate, crit rate is visible for Cyclone, and Trigger Rate, Spell Cooldown is on Ice Nova

As an addition, it would be very helpful to have a line where the required triggering skill attack rate is displayed to achieve the highest possible trigger rate using the current setup called something like "Optimal Triggering Skill Attack Rate"

Describe the solution you'd like

Since with one of the latest updates the spell description was updated to give a lot more info, I think it would be nice to modify it and expand it for visual clarity.

When a spell is triggered, It would be helpful to have additional info, currently in POB it looks like this:

  • CoC Triggering skill: Cyclone

followed by regular additional information

The line above is already colored to be highlighted and I think it could be expanded to include additional info on the spell's details page like this:

  • CoC Triggering skill: Cyclone
  • Triggering Skill Effective Crit Chance: x.xx%
  • Triggering Skill Hit rate: x%
  • Triggering Skill Attack Rate: x.xxs
  • "Optimal Triggering Skill Attack Rate" : x.xxs
  • Trigger Rate: x.xxs
  • Effective Trigger Rate: x.xxs
  • Spell Cooldown: x.xxs
Describe alternatives you've considered

Swapping between Cyclone and Ice Nova details and randomly adding/removing attack speed and crit to see when Trigger Rate is the highest with current cooldown reduction %, crit chance, hit chance, etc

Additional context

No response

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

この Issue にはファイル、テスト、コードのエントリーポイントが記載されていません。まずスペル詳細の表示箇所と、既存の Cast on Crit トリガー情報を見つけ、次に要求されているトリガー元スキルとクールダウンの値がどのように計算されているかを追跡します。トリガーされたスペルのビューに合意した追加の値が表示され、それらが現在の設定と一致すれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
lua
領域
desktop
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。