If the performance score command is given the -a flag, it is supposed to use the current online beatmap difficulty attributes. It does so by querying API, which queries osu-beatmap-difficulty-cache.
osu-tools is blissfully unaware of this, and as such attempts to deserialise the difficulty attributes returned from cache-then-web raw, which leads to it just using the default values (e.g. zeroes) for attributes that the difficulty lookup cache does not serialise out. This leads to obviously bogus results.
To fix, retrieve the APIBeatmap manually and do some (admittedly gnarly) grafting in order to produce a fully populated object. It's a bit ugly but it works locally without further changes so maybe fine?
The alternative would be to fix this on the beatmap difficulty cache, I guess. That said it's not like the cache/API returns these fields wrong either, it does correctly omit them, which is why I'm trying this first.
Found when investigating https://github.com/ppy/osu/issues/28006.
If the
performance scorecommand is given the-aflag, it is supposed to use the current online beatmap difficulty attributes. It does so by querying API, which queriesosu-beatmap-difficulty-cache.Problem is, that some of the difficulty attributes, namely the ones that are already present on
APIBeatmap, are not serialised out byosu-beatmap-difficulty-cache.osu-toolsis blissfully unaware of this, and as such attempts to deserialise the difficulty attributes returned from cache-then-web raw, which leads to it just using the default values (e.g. zeroes) for attributes that the difficulty lookup cache does not serialise out. This leads to obviously bogus results.To fix, retrieve the
APIBeatmapmanually and do some (admittedly gnarly) grafting in order to produce a fully populated object. It's a bit ugly but it works locally without further changes so maybe fine?The alternative would be to fix this on the beatmap difficulty cache, I guess. That said it's not like the cache/API returns these fields wrong either, it does correctly omit them, which is why I'm trying this first.