PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Minions with Multiple attack forms (like Agony Crawler) handled incorrectly for "Full DPS"

Ouverte
#6,265 0 commentaires 1 réaction 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

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

Description

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 calculation is supposed to be supported. If it isn't please open a feature request instead (Red text is a feature request).
What is the value from the calculation in-game?

Each of a minion's attack forms should be accounted for in its full dps calculation, including the (if known) timings or percentage of time it will use them in a standard situation. (IE, 70% slam, 30% roll).

What is the value from the calculation in Path of Building?

Minions with multiple attack forms require you to choose one attack form to be calculated for full dps. For example, Agony crawler has a Mortar, a Tail Spike, and a default attack. I can choose one of these while the skill is selected as my primary skill, and that attack form is used for Full DPS calculations. However, the minion uses its various attack forms automatically, so this never represents an accurate picture of the minion's full dps.

This is especially bad for builds that use many different minion types (Golems, Agony, Purity), as each of them is miscalculated.

How to reproduce the issue

No response

Character build code
https://pobb.in/O-8OMD8XFZV5
Screenshots

No response

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.

Piste de recherche

Commencez par charger le code de build de personnage lié et reproduisez le résultat Full DPS pour Agony Crawler et les autres minions ayant plusieurs formes d’attaque. Suivez le chemin de calcul existant des formes d’attaque des minions et déterminez comment les timings d’attaque ou les pourcentages d’utilisation peuvent être représentés ; le travail est considéré comme terminé lorsque Full DPS prend en compte chaque forme d’attaque applicable, plutôt que la seule forme sélectionnée.

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

Évaluation

Stack technique
lua
Domaine
game-dev
Type d'issue
Bug
Difficulté
5/5
Temps estimé
Plus d'une semaine
Activité
À l'abandon
Clarté
À clarifier
Accessibilité débutants
25/100

Recevez les nouvelles issues par e-mail

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