PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
FullDPS Corrupting Blood Incorrect Stack behaviour
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Lua
- Sterne
- 5.4k
- Forks
- 2.4k
- Ø Merge
- 1 T. 12 Std.
- Gemergte PRs (30 T.)
- 26
Beschreibung
After #6436 corrupting blood from multiple sources no longer stacks
Ingame there is a limit of 10 Corrupted Blood stacks, these can come from any mixture of sources, in PoB we only apply the source with the highest dps, regardless of how many leftover stacks it leaves
This for instance breaks multiple different sources of corrupting cry, or a mixture of corrupting cry with corrupting fever, it also breaks full dps count multipliers
The intended behavior is for full dps to add stacks of the highest source multiplied by its fulldps count, and then to only take the 10 highest stacks of it (eg if 8 corrupting cry and 4 corrupting fever stacks, and cry is higher than fever it should apply 8 from corrupting cry and only 2 from corrupting fever, or if corrupting cry count 4 on 2 warcries to allow both)
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Verfolge die Berechnung der Corrupting Blood-Stapel für FullDPS vollständig und vergleiche ihr Verhalten mit der Änderung in #6436. Überprüfe, dass Stapel aus mehreren Quellen und full-dps-Zählungen kombiniert und anschließend auf die 10 höchsten Stapel begrenzt werden, einschließlich der im Issue beschriebenen Beispiele mit gemischten Quellen.
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
- 42/100