PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
Add option to use active minion count for full DPS calculation
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
Check for duplicates
- I've checked for duplicate issues by using the search function of the issue tracker
Is your feature request related to a problem?
I would like to be able to use the Active Minion Limit as the minion count for the full dps calculation but there is no option to do so.
Describe the solution you'd like
In the skills menu to the right of the minion count box there should be a checkbox to automatically use the Active Minion Limit as the minion count.
This would make it easier to see dps increases from items that increase the minion limit as the increased limit would increase the total dps difference between them.
The same could be done for totems and any other skills that behave in a similar way.
Describe alternatives you've considered
No response
Additional context
No response
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
Beginnen Sie im Skills-Menü am Feld für die Minion-Anzahl und verfolgen Sie, wie Active Minion Limit und full DPS dargestellt werden. Bestätigen Sie, dass die angeforderte Option full DPS anhand des aktiven Limits aktualisiert, und ermitteln Sie anschließend, ob dasselbe Verhalten auch für Totems oder ähnliche Skills gelten sollte; der Payload nennt keine Dateien oder Tests.
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
- 42/100