PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Full DPS for multi-part skills

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

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

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

説明

Is your feature request related to a problem? Please describe.
The full DPS calculations currently does not include all components for some skills. For example, earthquake has both an initial impact and an aftershock, and both should be included in the calculations. Another example is divine ire's channeling and release.

Describe the solution you'd like
I would like the default to be that all components are included in the full dps calculation automatically, just like the duplication workaround described below.

Describe alternatives you've considered
My current workaround is to manually duplicate the skill group, and set the main skill on the sidebar so that both parts show up in the full DPS calculations.

image

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

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

はじめの一歩

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

調査の方向性

まず、完全な DPS 計算と、Earthquake や Divine Ire のような複数パートのスキルの処理を特定します。重複したスキルグループによって現在どのように両方のコンポーネントが表示されるのかを追跡し、その後、workaround なしですべてのコンポーネントが自動的に含まれることを確認します。

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

評価

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

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

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