PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Rallying cry and intimidating cry causing off hand damage to be more than mainhand

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

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

bug
主要言語
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
What is the expected behaviour/value?

Rallying cry and intimidating cry should have the same effect on main hand damage as they do offhand damage

What is the actual behaviour/value?

Enabling rallying cry or intimidating cry with identical weapons equiped causes off hand damage to be higher than main hand dmg. The increases/reductions and %more calculations under mainhand damage and off hand damage change to values that are not equal to each other.

How to reproduce the issue

Use chieftain tree in provided pob
0. Equip identical weapons to make comparison easier

  1. Disable rallying cry and intimidating cry
  2. Look at main hand and offhand damage in calc tab not
  3. Enable rallying cry and or intimidating cry
  4. Look at calc tab and see difference in main and off hand damage exists but is small
  5. Switch to berserker tree in provided pob and repeat steps 1-3
  6. Look at calc tab and see that offhand damage is now 1.5 times mainhand damage.
Build code
https://pastebin.com/t8qiPXJQ
Screenshots

No response

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

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

はじめの一歩

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

調査の方向性

まず、提供されたビルドコードを読み込み、同一の武器を使ってCalcタブの不一致を再現します。最初にChieftainツリーで、次にBerserkerツリーで行います。メインハンドとオフハンドのダメージについて、Rallying CryとIntimidating Cryの計算を追跡します。いずれかのクライを有効にしたとき、同一の武器がメインハンドとオフハンドに対して同等の効果を生じれば完了です。

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

評価

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

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

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