mirror of https://github.com/ppy/osu
Fix mania PP calculator applying incorrect score multiplier
This commit is contained in:
parent
03d3501a3c
commit
39a7bbdb9a
|
@ -43,14 +43,11 @@ public override PerformanceAttributes Calculate()
|
||||||
countMeh = Score.Statistics.GetValueOrDefault(HitResult.Meh);
|
countMeh = Score.Statistics.GetValueOrDefault(HitResult.Meh);
|
||||||
countMiss = Score.Statistics.GetValueOrDefault(HitResult.Miss);
|
countMiss = Score.Statistics.GetValueOrDefault(HitResult.Miss);
|
||||||
|
|
||||||
IEnumerable<Mod> scoreIncreaseMods = Ruleset.GetModsFor(ModType.DifficultyIncrease);
|
if (Attributes.ScoreMultiplier > 0)
|
||||||
|
{
|
||||||
double scoreMultiplier = 1.0;
|
// Scale score up, so it's comparable to other keymods
|
||||||
foreach (var m in mods.Where(m => !scoreIncreaseMods.Contains(m)))
|
scaledScore *= 1.0 / Attributes.ScoreMultiplier;
|
||||||
scoreMultiplier *= m.ScoreMultiplier;
|
}
|
||||||
|
|
||||||
// Scale score up, so it's comparable to other keymods
|
|
||||||
scaledScore *= 1.0 / scoreMultiplier;
|
|
||||||
|
|
||||||
// Arbitrary initial value for scaling pp in order to standardize distributions across game modes.
|
// Arbitrary initial value for scaling pp in order to standardize distributions across game modes.
|
||||||
// The specific number has no intrinsic meaning and can be adjusted as needed.
|
// The specific number has no intrinsic meaning and can be adjusted as needed.
|
||||||
|
|
Loading…
Reference in New Issue