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

Stun Duration Improvements #6228

Merged

Conversation

Regisle
Copy link
Member

@Regisle Regisle commented May 17, 2023

Minor improvements to enemy stun duration, adds support for "more stun duration" as well as double stun chance, and stun on crit, and of the chance mods use thier average value

it also shows the minimum and maximum duration if they are not the same (due to a chance mod)
image

@QuickStick123
Copy link
Contributor

There is this in regards to stun duration as well #5646, but I think there might be issues with certain mods that factor duration that wouldn't consider action speed if you just apply it directly to the duration. Just commenting in case it is something you want to work on as well while at it.

@Regisle
Copy link
Member Author

Regisle commented May 17, 2023

I think there might be issues with certain mods that factor duration that wouldn't consider action speed

This is why I didnt include it, effects like boneshatter do infact not care about it, however I could add it as a tooltip line I guess?

@QuickStick123
Copy link
Contributor

Yeah that might be a good compromise.

@QuickStick123 QuickStick123 added the enhancement New feature, calculation, or mod label May 17, 2023
@LocalIdentity LocalIdentity merged commit 2302668 into PathOfBuildingCommunity:dev Sep 9, 2023
@Regisle Regisle deleted the StunDurationImprovments branch September 9, 2023 04:07
shafouz pushed a commit to shafouz/PathOfBuilding that referenced this pull request Dec 8, 2023
* Stun Duration Improvements

* add action speed to breakdown
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature, calculation, or mod
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants