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

Enhanced Bipod stat penalties are displayed incorrectly #10479

Closed
Prometheus3375 opened this issue May 31, 2024 · 5 comments · Fixed by #10580
Closed

Enhanced Bipod stat penalties are displayed incorrectly #10479

Prometheus3375 opened this issue May 31, 2024 · 5 comments · Fixed by #10580

Comments

@Prometheus3375
Copy link

Prometheus3375 commented May 31, 2024

DIM Version

Version 8.21.1 (release), built on 27.05.2024, 20:49:58

Browser Details

Chrome 124.0.0.0

OS Details

Windows 10

Describe the bug

Variant w/o Bipod.
image

Variant w/ Bipod.
Blast radius: 95 -> 65 (correct)
Reload speed: 38 -> 18 (correct)
image

Variant w/ Enhanced Bipod.
Blast radius: 95 -> 40 (incorrect, should be 70)
Reload speed: 38 -> 3 (incorrect, should be 23)
image

It looks like when calculating penalties for Enhanced Bipod, penalties of regular Bipod are also included. 95 - (30 + 25) = 40 and 38 - (20 + 15) = 3.

Logs

No response

@bhollis
Copy link
Contributor

bhollis commented May 31, 2024

https://data.destinysets.com/i/InventoryItem:2282937672

Screenshot 2024-05-31 at 12 20 27 PM

So that's what Bungie says. isConditionallyActive of course is the trick - we can't know what the conditions are.

@Prometheus3375
Copy link
Author

I see, Light.gg also has this issue, but foundry displays it correctly, maybe they manually fixed that.

@bhollis
Copy link
Contributor

bhollis commented Jun 1, 2024

We could check all plugs and see if we can get away with a rule of "if the same stat is present twice, take the second occurrence"

This was referenced Jun 19, 2024
@bhollis
Copy link
Contributor

bhollis commented Jun 19, 2024

Hmm, this is a bit more complicated than expected. For example, here's Enhanced Lasting Impression: https://data.destinysets.com/i/InventoryItem:1167468626

Screenshot 2024-06-19 at 2 17 05 PM

It does appear to grant 55 blast radius.

@bhollis
Copy link
Contributor

bhollis commented Jun 19, 2024

And interestingly, Enhanced Bipod was fixed in the definitions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants