osu-tools icon indicating copy to clipboard operation
osu-tools copied to clipboard
verified publisher icon ppy

Lazer mods aren't being passed through to diffcalc

Open apollo-dw opened this issue 4 years ago • 2 comments

Commands such as dotnet run -- simulate osu 165967 -m BL don't work correctly at the moment, since mods are trimmed by LegacyHelper.

I'm getting around this locally at the moment by removing the relevant method. Smoogi seemed certain that this would break calculation however I can confirm that osu-tools functions just fine without the method? I can PR the linked branch however I really doubt that this is the correct way to solve this.

apollo-dw avatar Oct 09 '21 12:10 apollo-dw

Mods were trimmed as part of mania fixes PR https://github.com/ppy/osu-tools/pull/110

stanriders avatar Oct 09 '21 12:10 stanriders

This one’s actually the important one: https://github.com/ppy/osu-tools/pull/113 It has profile test cases in there to compare against. I’m fairly certain it still is required.

Perhaps it’s only the profile command that needs the trimming (it can also likely be done in a better way…).

smoogipoo avatar Oct 09 '21 12:10 smoogipoo