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

Negative MDEF should not increase damage from magic #3189

Closed
flamefury opened this issue Jun 7, 2018 · 1 comment
Closed

Negative MDEF should not increase damage from magic #3189

flamefury opened this issue Jun 7, 2018 · 1 comment
Labels
component:core A fault that lies within the main framework of rAthena mode:renewal A fault that exists within the renewal mode priority:low A fault that affects rAthena in one piece of functionality and is self-contained type:bug Issue that is a bug within rAthena

Comments

@flamefury
Copy link
Contributor

@aleos89 aleos89 added component:core A fault that lies within the main framework of rAthena priority:low A fault that affects rAthena in one piece of functionality and is self-contained mode:renewal A fault that exists within the renewal mode status:need more info Issue that needs more information from a creditable source type:bug Issue that is a bug within rAthena labels Jun 8, 2018
@flamefury
Copy link
Contributor Author

flamefury commented Jun 21, 2018

Some more proof without Assumptio:
AB soft MDEF = 41, hard MDEF = -18
Elder Willow MATK according to Divine-Pride: 43-52
Fire Bolt single hit damage: Low range of 2 to high range of 11

With 0 hard MDEF:

Damage = RANGE(43, 52) * (1000 + 0) / (1000 + 0 * 10) - 41
Damage = RANGE(43, 52) - 41
Damage = RANGE(2, 11)

If it was actually using -18 hMDEF, it would've looked like this instead:

Damage = RANGE(43, 52) * (1000 - 18) / (1000 - 180) - 41
Damage = RANGE(43, 52) * 982 / 820 - 41
Damage = RANGE(51, 62) - 41
Damage = RANGE(10, 21)

In other words, I should never see numbers 2-9 and should see numbers 12-21, but I see numbers in the low range and nothing higher than 11.

kRO replay file if you wish to check for equipment effects / shadow equipment.

@aleos89 aleos89 removed the status:need more info Issue that needs more information from a creditable source label Jul 16, 2018
SeravySensei pushed a commit to SeravySensei/rathena that referenced this issue Jan 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:core A fault that lies within the main framework of rAthena mode:renewal A fault that exists within the renewal mode priority:low A fault that affects rAthena in one piece of functionality and is self-contained type:bug Issue that is a bug within rAthena
Projects
None yet
Development

No branches or pull requests

2 participants