PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
Full DPS for multi-part skills
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
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.

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
Beginne damit, die vollständige DPS-Berechnung und die Behandlung mehrteiliger Skills wie Earthquake und Divine Ire zu lokalisieren. Verfolge, wie duplizierte Skill-Gruppen derzeit dazu führen, dass beide Komponenten angezeigt werden, und überprüfe anschließend, dass alle Komponenten automatisch ohne den Workaround einbezogen werden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- lua
- Bereich
- game-dev
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100