-
Notifications
You must be signed in to change notification settings - Fork 20
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
Discussion: Percent vs Flat values for stats that stack additively (critical strike chance, lifesteal, omnivamp) #54
Comments
Hello, as you said this would be a major change. We do not want to make breaking changes to the JSON. With either naming scheme you as the user have to know how to use the data. |
What about adding a Edit: On second thought, there are some issues with that solution since Riot doesn't handle flat Magic Penetration (additive) and percent Magic Penetration (multiplicative) as separate stats, the way they do for Lethality and Armor Penetration. In the JSON, you wouldn't be able to set a |
I'm wondering about the way Critical Strike Chance, Lifesteal, and Omnivamp are classified in item stats.
Each of these stats is displayed as a percentage in game. However, for the purposes of stat calculation, they behave more like flat bonuses; that is to say, they stack additively. This is in contrast to a stat like Armor Penetration which is both displayed as a percentage and stacks multiplicatively (and is also stored as a percent in the meraki data).
While the effects of these stats are indeed applied as percentages (e.g. 20% chance to critically strike or lifesteal healing for 8% of inflicted damage), the values are handled more like flat bonuses. Currently, meraki stores each of these three stats as percent values, but I think it would be more accurate to store them as flat values. To me, the difference between the way the stats stack seems more pertinent than the format in which they are displayed.
Obviously this would be a major change, so I wanted to ask some opinions on it.
The text was updated successfully, but these errors were encountered: