sim1222/osukey
1
1
Fork 0
mirror of https://github.com/osukey/osukey.git synced 2025-08-02 14:17:06 +09:00
Code Issues Packages Projects Releases Wiki Activity

Rework heatmap for more consistent performance

Browse Source
This commit is contained in:
smoogipoo
2020-06-19 19:08:36 +09:00
parent c9ad3192b0
commit cae3a5f447
2 changed files with 78 additions and 71 deletions
Download Patch File Download Diff File Expand all files Collapse all files

94
osu.Game.Rulesets.Osu/Statistics/Heatmap.cs
View File

@ -26,10 +26,15 @@ namespace osu.Game.Rulesets.Osu.Statistics
/// </summary>
private const float inner_portion = 0.8f;
private const float rotation = 45;
private const float point_size = 4;
/// <summary>
/// Number of rows/columns of points.
/// 4px per point @ 128x128 size (the contents of the <see cref="Heatmap"/> are always square). 1024 total points.
/// </summary>
private const int points_per_dimension = 32;
private Container<HitPoint> allPoints;
private const float rotation = 45;
private GridContainer pointGrid;
private readonly BeatmapInfo beatmap;
private readonly IReadOnlyList<HitEvent> hitEvents;
@ -111,52 +116,39 @@ namespace osu.Game.Rulesets.Osu.Statistics
}
}
},
allPoints = new Container<HitPoint>
pointGrid = new GridContainer
{
RelativeSizeAxes = Axes.Both
}
}
};
}
protected override void Update()
{
base.Update();
validateHitPoints();
}
Vector2 centre = new Vector2(points_per_dimension) / 2;
float innerRadius = centre.X * inner_portion;
private void validateHitPoints()
{
if (sizeLayout.IsValid)
return;
Drawable[][] points = new Drawable[points_per_dimension][];
allPoints.Clear();
// Since the content is fit, both dimensions should have the same size.
float size = allPoints.DrawSize.X;
Vector2 centre = new Vector2(size / 2);
int rows = (int)Math.Ceiling(size / point_size);
int cols = (int)Math.Ceiling(size / point_size);
for (int r = 0; r < rows; r++)
for (int r = 0; r < points_per_dimension; r++)
{
for (int c = 0; c < cols; c++)
points[r] = new Drawable[points_per_dimension];
for (int c = 0; c < points_per_dimension; c++)
{
Vector2 pos = new Vector2(c * point_size, r * point_size);
HitPointType pointType = HitPointType.Hit;
HitPointType pointType = Vector2.Distance(new Vector2(c, r), centre) <= innerRadius
? HitPointType.Hit
: HitPointType.Miss;
if (Vector2.Distance(pos, centre) > size * inner_portion / 2)
pointType = HitPointType.Miss;
allPoints.Add(new HitPoint(pos, pointType)
var point = new HitPoint(pointType)
{
Size = new Vector2(point_size),
Colour = pointType == HitPointType.Hit ? new Color4(102, 255, 204, 255) : new Color4(255, 102, 102, 255)
});
};
points[r][c] = point;
}
}
pointGrid.Content = points;
if (hitEvents.Count > 0)
{
// Todo: This should probably not be done like this.
@ -170,41 +162,39 @@ namespace osu.Game.Rulesets.Osu.Statistics
AddPoint(((OsuHitObject)e.LastHitObject).StackedEndPosition, ((OsuHitObject)e.HitObject).StackedEndPosition, e.CursorPosition.Value, radius);
}
}
sizeLayout.Validate();
}
protected void AddPoint(Vector2 start, Vector2 end, Vector2 hitPoint, float radius)
{
if (allPoints.Count == 0)
if (pointGrid.Content.Length == 0)
return;
double angle1 = Math.Atan2(end.Y - hitPoint.Y, hitPoint.X - end.X); // Angle between the end point and the hit point.
double angle2 = Math.Atan2(end.Y - start.Y, start.X - end.X); // Angle between the end point and the start point.
double finalAngle = angle2 - angle1; // Angle between start, end, and hit points.
float normalisedDistance = Vector2.Distance(hitPoint, end) / radius;
// Since the content is fit, both dimensions should have the same size.
float size = allPoints.DrawSize.X;
// Convert the above into the local search space.
Vector2 localCentre = new Vector2(points_per_dimension) / 2;
float localRadius = localCentre.X * inner_portion * normalisedDistance; // The radius inside the inner portion which of the heatmap which the closest point lies.
double localAngle = finalAngle + 3 * Math.PI / 4; // The angle inside the heatmap on which the closest point lies.
Vector2 localPoint = localCentre + localRadius * new Vector2((float)Math.Cos(localAngle), (float)Math.Sin(localAngle));
// Find the most relevant hit point.
double minDist = double.PositiveInfinity;
HitPoint point = null;
foreach (var p in allPoints)
for (int r = 0; r < points_per_dimension; r++)
{
Vector2 localCentre = new Vector2(size / 2);
float localRadius = localCentre.X * inner_portion * normalisedDistance;
double localAngle = finalAngle + 3 * Math.PI / 4;
Vector2 localPoint = localCentre + localRadius * new Vector2((float)Math.Cos(localAngle), (float)Math.Sin(localAngle));
float dist = Vector2.Distance(p.DrawPosition + p.DrawSize / 2, localPoint);
if (dist < minDist)
for (int c = 0; c < points_per_dimension; c++)
{
minDist = dist;
point = p;
float dist = Vector2.Distance(new Vector2(c, r), localPoint);
if (dist < minDist)
{
minDist = dist;
point = (HitPoint)pointGrid.Content[r][c];
}
}
}
@ -216,11 +206,11 @@ namespace osu.Game.Rulesets.Osu.Statistics
{
private readonly HitPointType pointType;
public HitPoint(Vector2 position, HitPointType pointType)
public HitPoint(HitPointType pointType)
{
this.pointType = pointType;
Position = position;
RelativeSizeAxes = Axes.Both;
Alpha = 0;
}

55
osu.Game.Tests/Visual/Ranking/TestSceneAccuracyHeatmap.cs
View File

@ -2,11 +2,13 @@
// See the LICENCE file in the repository root for full licence text.
using System.Collections.Generic;
using NUnit.Framework;
using osu.Framework.Extensions.Color4Extensions;
using osu.Framework.Graphics;
using osu.Framework.Graphics.Containers;
using osu.Framework.Graphics.Shapes;
using osu.Framework.Input.Events;
using osu.Framework.Threading;
using osu.Framework.Utils;
using osu.Game.Beatmaps;
using osu.Game.Rulesets.Osu;
@ -20,13 +22,18 @@ namespace osu.Game.Tests.Visual.Ranking
{
public class TestSceneAccuracyHeatmap : OsuManualInputManagerTestScene
{
private readonly Box background;
private readonly Drawable object1;
private readonly Drawable object2;
private readonly TestHeatmap heatmap;
private Box background;
private Drawable object1;
private Drawable object2;
private TestHeatmap heatmap;
private ScheduledDelegate automaticAdditionDelegate;
public TestSceneAccuracyHeatmap()
[SetUp]
public void Setup() => Schedule(() =>
{
automaticAdditionDelegate?.Cancel();
automaticAdditionDelegate = null;
Children = new[]
{
background = new Box
@ -41,7 +48,7 @@ namespace osu.Game.Tests.Visual.Ranking
},
object2 = new BorderCircle
{
Position = new Vector2(500, 300),
Position = new Vector2(100, 300),
},
heatmap = new TestHeatmap(new TestBeatmap(new OsuRuleset().RulesetInfo).BeatmapInfo, new List<HitEvent>())
{
@ -50,22 +57,32 @@ namespace osu.Game.Tests.Visual.Ranking
Size = new Vector2(130)
}
};
});
[Test]
public void TestManyHitPointsAutomatic()
{
AddStep("add scheduled delegate", () =>
{
automaticAdditionDelegate = Scheduler.AddDelayed(() =>
{
var randomPos = new Vector2(
RNG.NextSingle(object1.DrawPosition.X - object1.DrawSize.X / 2, object1.DrawPosition.X + object1.DrawSize.X / 2),
RNG.NextSingle(object1.DrawPosition.Y - object1.DrawSize.Y / 2, object1.DrawPosition.Y + object1.DrawSize.Y / 2));
// The background is used for ToLocalSpace() since we need to go _inside_ the DrawSizePreservingContainer (Content of TestScene).
heatmap.AddPoint(object2.Position, object1.Position, randomPos, RNG.NextSingle(10, 500));
InputManager.MoveMouseTo(background.ToScreenSpace(randomPos));
}, 1, true);
});
AddWaitStep("wait for some hit points", 10);
}
protected override void LoadComplete()
[Test]
public void TestManualPlacement()
{
base.LoadComplete();
Scheduler.AddDelayed(() =>
{
var randomPos = new Vector2(
RNG.NextSingle(object1.DrawPosition.X - object1.DrawSize.X / 2, object1.DrawPosition.X + object1.DrawSize.X / 2),
RNG.NextSingle(object1.DrawPosition.Y - object1.DrawSize.Y / 2, object1.DrawPosition.Y + object1.DrawSize.Y / 2));
// The background is used for ToLocalSpace() since we need to go _inside_ the DrawSizePreservingContainer (Content of TestScene).
heatmap.AddPoint(object2.Position, object1.Position, randomPos, RNG.NextSingle(10, 500));
InputManager.MoveMouseTo(background.ToScreenSpace(randomPos));
}, 1, true);
AddStep("return user input", () => InputManager.UseParentInput = true);
}
protected override bool OnMouseDown(MouseDownEvent e)