project-slippi / slippi-js

Parse slp files and compute stats
GNU Lesser General Public License v3.0
148 stars 79 forks source link

Fix edge-case stats calculation #57

Closed OGoodness closed 3 years ago

OGoodness commented 3 years ago

This PR fixes two critical issues with stats calculation:

  1. command grabs are not included in conversions
  2. total damage calculation is too naive and does not account for damage recovery, pichu's self damage, and blast zone damage

New Slippi files are included for tests including: