PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Minions with Multiple attack forms (like Agony Crawler) handled incorrectly for "Full DPS"

Offen
#6,265 0 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

bug: calculation
Vorherrschende Sprache
Lua
Sterne
5.4k
Forks
2.4k
Ø Merge
1 T. 12 Std.
Gemergte PRs (30 T.)
26

Beschreibung

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 calculation is supposed to be supported. If it isn't please open a feature request instead (Red text is a feature request).
What is the value from the calculation in-game?

Each of a minion's attack forms should be accounted for in its full dps calculation, including the (if known) timings or percentage of time it will use them in a standard situation. (IE, 70% slam, 30% roll).

What is the value from the calculation in Path of Building?

Minions with multiple attack forms require you to choose one attack form to be calculated for full dps. For example, Agony crawler has a Mortar, a Tail Spike, and a default attack. I can choose one of these while the skill is selected as my primary skill, and that attack form is used for Full DPS calculations. However, the minion uses its various attack forms automatically, so this never represents an accurate picture of the minion's full dps.

This is especially bad for builds that use many different minion types (Golems, Agony, Purity), as each of them is miscalculated.

How to reproduce the issue

No response

Character build code
https://pobb.in/O-8OMD8XFZV5
Screenshots

No response

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, den verknüpften Charakter-Build-Code zu laden und das Full-DPS-Ergebnis für den Agony Crawler und andere Minions mit mehreren Angriffsformen zu reproduzieren. Verfolge den bestehenden Berechnungspfad für Minion-Angriffsformen und ermittle, wie Angriffstaktungen oder Nutzungsprozentsätze dargestellt werden können; als abgeschlossen gilt die Aufgabe, wenn Full DPS jede zutreffende Angriffsform berücksichtigt und nicht nur die ausgewählte.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
lua
Bereich
game-dev
Issue-Typ
Bug
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.