PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
Ramp Time in Side Bar
Personne n'a encore pris cette issue.
- Langage dominant
- Lua
- Étoiles
- 5.4k
- Forks
- 2.4k
- Merge moyen
- 1 j 12 h
- PR mergées (30 j)
- 26
Description
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
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Aucun fichier ni test n’est indiqué. Commencez par localiser l’affichage du mode de calcul dans la barre latérale ainsi que les statistiques existantes poison, impale et total-damage-per-poison ; comparez les formules de ramp time demandées aux calculs actuels. La tâche est terminée lorsque les modes de DPS combinant hit et poison/impale affichent la ramp time sans surcharger la barre latérale.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- lua
- Domaine
- desktop
- Type d'issue
- Fonctionnalité
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100