PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Lancing Steel damage calculation error

Aperta
#9,467 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

bug
Lingua principale
Lua
Stelle
5.4k
Fork
2.4k
Merge medio
1g 12h
PR unite (30g)
26

Descrizione

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 open and closed issues by using the search function of the issue tracker
What platform are you running Path of Building on?

Windows

How is Path of Building expected to behave?

Damage calculation for Lancing Steel should not display errors.

How does Path of Building behave?

Lancing Steel displays a red "nan" output for damage calculations when setting the Skill Part to "All Projectiles Hit."
Same issue occurs for Lancing Steel of Spraying.

How to reproduce the issue
  1. Equip any sword.
  2. Add Lancing Steel.
  3. Switch Skill Part from "Single Projectile Hit" to "All Projectiles Hit."
PoB for PoE1 build code
https://pobb.in/T9NePz1hNi07
Screenshots
Image

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Inizia riproducendo il problema con la build PoB collegata, usando Lancing Steel e la parte dell'abilità "All Projectiles Hit"; controlla anche Lancing Steel of Spraying. Traccia il calcolo dei danni per queste parti dell'abilità e identifica perché produce un valore "nan" in rosso. Il lavoro è completato quando entrambe le abilità mostrano calcoli dei danni validi per la configurazione descritta.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
lua
Ambito
backend
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.