PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Dark pact of thrarthus doesn't take into account if you use traps/mines or not

Offen
#9,428 0 Kommentare 0 Reaktionen 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 open and closed 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 platform are you running Path of Building on?

Windows

What is the value from the calculation in-game?

I don't have the in game calculations, but i checked the code, and that is calculating ruin stacks, which do not work when you have traps (max is 1 on the trap) this causes the average to appear as if you're dealing 100 mill dps while in fact you'd do 1 million

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

i checked the code, and that is calculating ruin stacks, which do not work when you have traps (max is 1 on the trap) this causes the average to appear as if you're dealing 100 mill dps while in fact you'd do 1 million

How to reproduce the issue

easiest value to see this change with is the quality setting on thrarthus, this adds damage eventhough it only allows you to stack ruin more quickly. nothing else.
I believe adding another setting along with average and max ruin. would do it
Part 1 = No Ruin
Part 2 = Average
Part 3 = Max Ruin

PoB for PoE1 build code
https://pobb.in/d8v0vUo1Z8Dz
Screenshots
Image Image

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

Reproduziere das Problem mit dem verlinkten PoB für den PoE1-Build-Code und verfolge anschließend die Berechnung der ruin stacks sowie deren Behandlung von Fallen und Minen. Prüfe, wie sich die Qualitätseinstellung von Thrarthus auf den angezeigten Schaden auswirkt; abgeschlossen ist die Aufgabe, wenn der Schaden von Fallen und Minen nicht mehr von uneingeschränkten ruin stacks ausgeht und die angeforderten ruin-state choices korrekt dargestellt werden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
lua
Bereich
game-dev
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
38/100

Neue Issues direkt in Ihr Postfach

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