PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Mirage Archer and Barrage interact to Create Incorrect Tooltips

Offen
#2,680 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

bug
Vorherrschende Sprache
Lua
Sterne
5.4k
Forks
2.4k
Ø Merge
1 T. 12 Std.
Gemergte PRs (30 T.)
26

Beschreibung

  • What happened?
    DPS Delta tooltips for activating support gems or added damage auras were incorrect while both Mirage archer and Barrage Support were linked to the main skill, but were correct when either of those supports were deactivated.
  • What were you trying to do?
    Decide which auras and support gems to use.
  • What steps will reproduce the bug?
    Open this pastebin: https://pastebin.com/YUxmxiXj
    -Hover over the "Enabled" checkbox for Added Lightning Damage or Herald of Thunder
    -Observe that in the hover tooltip "Mirage DPS" % gain is much higher than the Total DPS % gain, when they should be identical +/- rounding
    -Actually check one of those boxes, observe that the combined DPS increased by less than the tooltip said it would (reflecting the correct gain in Mirage Archer damage)
    -De-activate either Mirage Archer or Barrage Support and repeat above, note that there is no such issue.
  • Does it reproduce every time?
    Yes
  • Provide a build code:
    https://pastebin.com/YUxmxiXj

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, den verlinkten Pastebin-Build zu laden und die Tooltip-Abweichung zu reproduzieren, während Mirage Archer und Barrage Support beide aktiv sind. Vergleiche die Hover-Vorschau für Added Lightning Damage oder Herald of Thunder mit der tatsächlichen DPS-Änderung und wiederhole dies, nachdem jeweils eine der beiden Supports deaktiviert wurde. Als erledigt gilt es, wenn Mirage DPS und Total DPS denselben Zugewinn anzeigen, abgesehen von Rundungsabweichungen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
lua
Bereich
desktop, game-dev
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
42/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.