PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
Ramp Time in Side Bar
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
I'd like ramp time to be included whenever the calculation mode combines hit dps with poison/impale dps. It's important context for the final dps number that players can miss when talking about X bajillion viper strike dps.
If the box is getting too crowded then maybe replace the "total damage per poison" stat, which doesn't seem very relevant when dps is concerned.
ramp time for poison dps = poison duration
ramp time for impale dps = max number of impales / attacks per second
Originally posted by @Sectoidfodder in https://github.com/PathOfBuildingCommunity/PathOfBuilding/pull/158#issuecomment-594084168
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
Es ist keine Datei oder kein Test genannt. Beginne damit, die Anzeige des Berechnungsmodus in der Seitenleiste sowie die vorhandenen poison-, impale- und total-damage-per-poison-Stats zu finden; vergleiche die angeforderten Formeln für die ramp time mit den aktuellen Berechnungen. Die Aufgabe ist abgeschlossen, wenn die kombinierten Hit-plus-poison/impale-DPS-Modi die ramp time anzeigen, ohne die Seitenleiste zu überladen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- lua
- Bereich
- desktop
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100