selicia / splat2-calc

Gear planning tool for Splatoon 2.
MIT License
14 stars 5 forks source link

Rapid Blaster/Rapid Blaster Pro MPU(Damage Radius) #21

Closed nikuD29 closed 5 years ago

nikuD29 commented 5 years ago

The effect caps at 29AP instead of 57AP.

selicia commented 5 years ago

Thanks for the heads-up. I'll include a fix for this in the next release.

selicia commented 5 years ago

Rapid Blaster & Rapid Blast Pro MPU Caps

selicia commented 5 years ago

Looked into this some more. I wasn't aware there was a cap on the damage radius other than the max AP modifier. You're right that ~29 AP is approximately hitting the cap. I'll update the code so it uses the base Damage Radius values and strictly applies the modifier calculated from the AP with a check for the cap. Will take a little bit more work than I originally thought. Are you aware of any other weapons that have MPU stat caps?

Example data taken from Lean's data below:

CollisionRadiusMiddle: 33
CollisionRadiusMiddle_MWPUG_Max: 35
CollisionRadiusMiddleRate_MWPUG_High: 1.093939
CollisionRadiusMiddleRate_MWPUG_Mid: 1.036364
nikuD29 commented 5 years ago

As far as I know, this is the only non-Damage-Up MPU that has a cap. I also found L3 MPU not displaying when I was checking it through.

selicia commented 5 years ago

4adfa20a72015c9f82839d81660d3b564ad3f43a Fixes the Rapid Blaster & Rapid Blaster Pro MPU caps. 58cd4da19bcd7ccf9e4bc419e27f750ad308f0c9 Fixes the L-3 Nozzelnose bug that was preventing MPU from displaying.

selicia commented 5 years ago

Fixes are live!