PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Destrutive Link Gives incorrect Critical strike chance

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

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

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

説明

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
Check for support
  • I've checked that the behaviour is supposed to be supported. If it isn't please open a feature request instead (Red text is a feature request).
What is the behaviour in-game?

Destructive Link gives the players final critical strike chance of a basic attack with the mainhand weapon after modifiers to the linked target

What is the behaviour in Path of Building?

Only gives the mainhand weapons base critical strike chance, and does not take into account other modifiers like power charges or crit from tree

Additional Info

#6139 added support for destructive links critical strike chance mod but does not properly give correct crit chance, this is fine for how most people use it (with Trypanon) but can be misleading, it wasnt properly support due to not having the calculated crit chance, as this would require an extra calc pass (like fulldps) of a basic attack to get the correct value taking into account buffs etc

This is relatively low priority to fix though due to Trypanon being the main use of the link

How to reproduce the issue

No response

Character build code

No response

Screenshots

No response

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

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

はじめの一歩

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

調査の方向性

まず issue #6139 と、Destructive Link における既存のクリティカルストライク処理を確認してください。基本攻撃の計算が現在どのようにメインハンド武器の基礎確率を取得しているかを追跡し、次に、計算パスで power charges、パッシブツリーの修正値、その他の buffs を含める方法を判断してください。完了の条件は、リンクされた対象の最終クリティカルストライク確率と一致することです。

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

評価

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

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

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