General's Cry Incorrect Behavior with Blade Flurry

Ouverte
#5,935 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
4/5
Temps estimé
3-5 jours
Accessibilité débutants
35/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
À l'abandon
Stack technique
lua
Domaine
game-dev

Piste de recherche

Commencez dans le chemin Calcs pour General's Cry et Blade Flurry, reproduisez le Skill DPS gonflé avec le build de personnage lié et suivez la façon dont les étapes de canalisation et la vitesse d'attaque sont appliquées. Le travail est terminé lorsque la limite de canalisation de 0,6 seconde et la limite d'une attaque affectent le DPS, tandis que le tooltip de vitesse d'attaque reste inchangé.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

bug: behaviour
Check version
  • I'm running the latest version of Path of Building and I've verified this by checking the changelog
Check for duplicates
  • I've checked for duplicate issues by using the search function of the issue tracker
Check for support
  • I've checked that the behaviour is supposed to be supported. If it isn't please open a feature request instead (Red text is a feature request).
What is the behaviour in-game?

Mirage Warriors will either:

  • Channel for up to 0.6sec -> release -> disappear (0.6sec channel time cap)
  • Channel up to 6 stacks -> release -> disappear (1 attack cap)
What is the behaviour in Path of Building?

It seems that Path of Building ignores the 0.6sec channel time cap and 1 max attack cap for Blade Flurry, instead it multiplies the average damage by the attack rate. This leads to a highly inflated DPS.

I suggest the following solution:

  • Cap the maximum number of stages providing a bonus to effective_max_stages = min(floor(attack_rate * general_cry_max_channel_time), base_max_stages).
  • Multiply the DPS by effective_max_stages/attack_rate to normalize it over 1 second, i.e. get the total damage done per 0.6sec of channeling, similar to non-channeled skills.

This way we can get a much more accurate read on the expected damage, without removing/modifying the tooltip about the attack rate, which is important for gearing, and the only remaining step to get the actual DPS is isolated to managing the warcry cooldown and spawning rate of the warriors.

How to reproduce the issue
  1. Add General's Cry, Blade Flury and equip a valid weapon
  2. Go to Calcs, select and highlight Skill DPS
Character build code
https://pastebin.com/L449UAZX
Screenshots

GeneralsCry

Langage dominant
Lua
Étoiles
5.4k
Forks
2.4k
Merge moyen
1 j 12 h
PR mergées (30 j)
26

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de PathOfBuildingCommunity/PathOfBuilding

Toutes les issues de PathOfBuildingCommunity/PathOfBuilding

Issues similaires

Plus d'issues Lua

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.