umang-p / brainlets

A website for tools/utilities related to the Girls' Frontline mobile game. http://brainlets.moe
http://brainlets.moe
GNU General Public License v3.0
6 stars 12 forks source link

K2 damage calculation does not appear to be correct when armor is involved #20

Closed AristocratMC closed 5 years ago

AristocratMC commented 5 years ago

(I'll probably get to this in a few days if you are busy, leaving issue here so I don't lose track.)

Reproduction steps:

Expected result:

Actual result:

Analysis: At the very least, we know the damage calculation for this unit in particular is only calculating armor once. However this doesn't explain the other anomaly of only dealing 10 damage per shot even though K2 does a triple shot and thus should deal 15.

To-Do:

umang-p commented 5 years ago

Commit https://github.com/umang-p/brainlets/commit/40a1313276031c9bcf4bd5939c699d1fa93246de will fix her damage against high armor enemies, letting her deal 15 damage per shot. However, taking crit into account will add a multiplier of 1.34 with default equips, so she will be shown as doing 20 damage per shot. I am unaware of whether or not the game rounds up or down when a critical hit occurs at minimum damage so for now I will leave it as it is.

Leaving this issue open as I believe there is a problem with her skill switching modes upon cooldown. (will investigate further tomorrow)

umang-p commented 5 years ago

Turns out there was an issue when switching from note to fever mode. Fixed in 9d9442c50d76299b83c86676f4cf95b97fac79ff

K2's skill should function properly now so I'm closing this. Please open another issue if you find any problems.