ppy / osu-tools

command line tools to get stuff done
MIT License
105 stars 78 forks source link

Fix `performance legacy-score` command doing things it doesn't need to #205

Closed bdach closed 5 months ago

bdach commented 5 months ago

I'm not exactly sure when or how this happened because I tested this previously (did some web change cause this?), but the performance legacy-score command is:

while all it needs to be doing is reading what it is given from the API. All of the data it needs is already there, in a form ready for consumption.

For instance, see output of performance legacy-score 215127587 2 (see relevant score):

master ``` Basic score info ────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────── beatmap : 2062131 - Ricky Montgomery - Snow (Crowley) total score : 1071818 legacy total score : 1117705 accuracy : 99.37 combo : 1015 mods : DT, HD, CL, CL Hit statistics ────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────── great : 985 largetickhit : 30 smalltickhit : 867 smalltickmiss : 12 Performance attributes ────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────── pp : 373.07 Difficulty attributes ────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────── star rating : 5.49 max combo : 1,015.00 approach rate : 9.87 ```
this PR ``` Basic score info ────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────── beatmap : 2062131 - Ricky Montgomery - Snow (Crowley) total score : 1117518 legacy total score : 33334515 accuracy : 99.37 combo : 1015 mods : DT, HD, CL Hit statistics ────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────── great : 985 largetickhit : 30 smalltickhit : 867 smalltickmiss : 12 Performance attributes ────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────── pp : 373.07 Difficulty attributes ────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────── star rating : 5.49 max combo : 1,015.00 approach rate : 9.87 ```