Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DPS of characters are displayed wrong since the last update #4563

Closed
2 tasks done
Sallazarowns opened this issue Jul 12, 2022 · 10 comments
Closed
2 tasks done

DPS of characters are displayed wrong since the last update #4563

Sallazarowns opened this issue Jul 12, 2022 · 10 comments
Labels
bug Something isn't working

Comments

@Sallazarowns
Copy link

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

What is the expected behaviour/value?

My 2 builds showed 65+ million and 41+ million DPS respectively.

What is the actual behaviour/value?

After update the values don't go over 11 million

How to reproduce the issue

  1. Import skills, jewels, items and passive tree
  2. Go to skills tab
  3. Select all skills categories into FULL DPS

Character build code

https://pastebin.com/vsS2SFak - cyclone slayer (showed 65 million full DPS)
https://pastebin.com/MBGyGb7n - tornado shot deadeye (showed 41 million full DPS)

Screenshots

No response

@Sallazarowns Sallazarowns added the bug Something isn't working label Jul 12, 2022
@QuickStick123
Copy link
Contributor

This is because of the new Uber Pinnacle Boss config option which applies 70% less damage taken to enemies.
image
Setting to guardian/pinnacle resolves this issue.
image

@QuickStick123 QuickStick123 added invalid This doesn't seem right and removed bug Something isn't working labels Jul 12, 2022
@Sallazarowns
Copy link
Author

Sallazarowns commented Jul 12, 2022

Haven't noticed that change. Sorry.

Some DPS is still missing, but I guess that's the new way of calculating the damage reduction of the boss? I used to run with Sirus A9 iirc.

EDIT: Ignore this, some of the buffs weren't checked in the config tab. Again sorry for the bothering, and thanks for the awesome tool you've created for us to play with <3

@adracea
Copy link

adracea commented Jul 12, 2022

I'm pretty sure the issue should still be active. I'm seeing incredibly lower numbers on Righteous Fire DoT builds. Drops from 6 mil to under 1 mil. The issues seems to be that some player applied debuffs don't do anything (fire exposure for example don't do anything to fire trap dps).

@deathbeam
Copy link
Contributor

You need to switch from uber pinnacle boss to guardian/pinnacle boss in config:

image

@Lothrik
Copy link
Contributor

Lothrik commented Jul 12, 2022

I'm pretty sure the issue should still be active. I'm seeing incredibly lower numbers on Righteous Fire DoT builds. Drops from 6 mil to under 1 mil. The issues seems to be that some player applied debuffs don't do anything (fire exposure for example don't do anything to fire trap dps).

I tested applying Exposure through Wave of Conviction, Scorching Ray, and Elemental Equlibrium. All of them work, you probably just have something configured incorrectly.

@adracea
Copy link

adracea commented Jul 12, 2022

image

Side by side comparison of the same build loaded in 2.18.1 vs 2.19.0

Edit: POB : https://pastebin.com/w737Fn1j

Notable is the fact that even vs normal enemies(not a boss setting), the DPS shown for both RF and firetrap is LOWER than the in-game dps displayed on the tooltip.

@adracea
Copy link

adracea commented Jul 12, 2022

image
image

Left side is 2.19 right side is 2.18.1
The issue seems to be coming from the calculation of the "Skill damage over time" I believe.

@Lothrik
Copy link
Contributor

Lothrik commented Jul 12, 2022

It looks like the base DoT damage for Fire Trap was broken by a recent change to stat interpolation (it went from 4824.8 to 289.5 at gem level 21). We'll look into fixing it.

Edit: #4575 addresses the DoT damage issue.

@LocalIdentity
Copy link
Contributor

Fixed by #4581 and #4574

@LocalIdentity LocalIdentity added bug Something isn't working and removed invalid This doesn't seem right labels Jul 13, 2022
@QuickStick123
Copy link
Contributor

Fixed in most recent release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

6 participants