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

Improve handling of unscalable mods and fix some mods from alternate qualities not applying. #4906

Merged
merged 13 commits into from
Sep 14, 2022

Conversation

Paliak
Copy link
Contributor

@Paliak Paliak commented Aug 18, 2022

Fixes #4848 #4849 #5031 #5042

Absorbed #5043.

This pr changes the way unscaled mods are handled and fixes related issues adding unscaled tag to a variety of mods. Also fixes some mods from alternate qualities not applying correctly and implements parsing for a few currently unhandled stats.

Before screenshot:

obraz

After screenshot:

obraz

@Paliak
Copy link
Contributor Author

Paliak commented Aug 18, 2022

Breakdown for impurities currently broken. Working on a fix. Fixed

@QuickStick123 QuickStick123 added the bug: behaviour Behavioral differences label Aug 19, 2022
@Nostrademous
Copy link
Contributor

@Paliak why couldn't 0, 0, { type = "GlobalEffect", effectType = "Aura", unscalable = true } part be added at the SkillStatMap.lua level to all the mods?

@Paliak
Copy link
Contributor Author

Paliak commented Aug 20, 2022

I looked at how purity of elements worked and applied that to the impurities.

Paliak added 2 commits August 20, 2022 18:55
…dummy display stat. Re add entries in skillStatMap in case they're used by something.
QuickStick123 and others added 2 commits September 6, 2022 14:22
* Fix Up several other instances of this and add support for a couple aura mods while at it.

* FIX: add a check in case (env.configInput["enemyCritChance"] ends up nil.

Co-authored-by: Paliak <[email protected]>
@Paliak Paliak changed the title FIX: fix binary mods from auras scaling with aura effect FIX: fix binary mods from auras scaling with aura effect and improve handling of unscaled mods. Sep 6, 2022
@Paliak Paliak changed the title FIX: fix binary mods from auras scaling with aura effect and improve handling of unscaled mods. Improve handling of unscalable mods and fix some mods from alternate qualities not applying. Sep 6, 2022
@QuickStick123
Copy link
Contributor

Fixes #5031

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug: behaviour Behavioral differences
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Vaal Clarity and potnetially other auras set stats scale with aura effect.
4 participants