mirror of
https://github.com/ppy/osu.git
synced 2024-11-18 05:12:53 +08:00
5fde4f2c0c
Changes to lifetime calculation in scrolling rulesets introduced in #7367, which aimed to account for the distance between hit objects' origin and its edge entering the scrolling area, fixed some issues with hitobjects appearing abruptly, but also regressed some other scenarios. Upon investigation, the regression was localised to the overlapping scroll algorithm. The reason for this was two-fold: * The previous code used TimeAt() to calculate the time of travel from the hit object's edge to its origin. For other algorithms, that time can be accurately reconstructed, because they don't have periods of time where there are multiple hit objects scrolling at different velocities. That invariant does not hold for the overlapping algorithm, therefore it is possible for different values to be technically correct for TimeAt(). However, the only value that matters for the adjustment is the one that's indicated by the control point that applies to the hit object origin, which can be uniquely identified. * Additionally, the offset returned (even if correct) was applied externally to the hit object's start time and passed to GetDisplayStartTime(). In the overlapping algorithm, the choice of control point used in GetDisplayStartTime() is important, since the value of the speed multiplier is read within. Externally rewinding the hit object's start time meant that in some cases the speed multiplier of the *previous* control point is applied, which led to hit objects appearing too late if the scrolling rate decreased. Because of the above, modify GetDisplayStartTime() to take the offset into account in all algorithms, and apply the adjustment correctly inside of them. The constant and sequential algorithms needed no adjustment from the previous logic, since: * the constant algorithm disregarded control points, and * the sequential algorithm would effectively rewind to time = 0, calculate the absolute distance from time = 0 to the hit object start, apply the origin offset *to the absolute distance*, and then convert back to time, applying all control points in sequence. Due to this it was impossible for control points to get mixed up while calculating. As for the overlapping algorithm, the high-level logic is as follows: * The distance that the origin has to travel is the length of the scroll plus the distance from the origin to the object edge. * The above distance divided by the scroll length gives the relative scroll lengths that the object has to travel. * As one relative scroll length takes one time range, the relative travel length multiplied by the time range gives the absolute travel time of the object origin. * Finally, the control point multiplier applicable at origin time is applied to the whole travel time. Correctness of the above is demonstrated by visual tests added before and headless unit tests of the algorithms themselves. The sequential scroll algorithm was not covered by unit tests, and remains uncovered due to floating-point inaccuracies that should be addressed separately.
122 lines
5.4 KiB
C#
122 lines
5.4 KiB
C#
// Copyright (c) ppy Pty Ltd <contact@ppy.sh>. Licensed under the MIT Licence.
|
|
// See the LICENCE file in the repository root for full licence text.
|
|
|
|
using System;
|
|
using System.Collections.Generic;
|
|
using osu.Game.Rulesets.Timing;
|
|
|
|
namespace osu.Game.Rulesets.UI.Scrolling.Algorithms
|
|
{
|
|
public class SequentialScrollAlgorithm : IScrollAlgorithm
|
|
{
|
|
private readonly Dictionary<double, double> positionCache;
|
|
|
|
private readonly IReadOnlyList<MultiplierControlPoint> controlPoints;
|
|
|
|
public SequentialScrollAlgorithm(IReadOnlyList<MultiplierControlPoint> controlPoints)
|
|
{
|
|
this.controlPoints = controlPoints;
|
|
|
|
positionCache = new Dictionary<double, double>();
|
|
}
|
|
|
|
public double GetDisplayStartTime(double originTime, float offset, double timeRange, float scrollLength)
|
|
{
|
|
double adjustedTime = TimeAt(-offset, originTime, timeRange, scrollLength);
|
|
return adjustedTime - timeRange - 1000;
|
|
}
|
|
|
|
public float GetLength(double startTime, double endTime, double timeRange, float scrollLength)
|
|
{
|
|
var objectLength = relativePositionAtCached(endTime, timeRange) - relativePositionAtCached(startTime, timeRange);
|
|
return (float)(objectLength * scrollLength);
|
|
}
|
|
|
|
public float PositionAt(double time, double currentTime, double timeRange, float scrollLength)
|
|
{
|
|
// Caching is not used here as currentTime is unlikely to have been previously cached
|
|
double timelinePosition = relativePositionAt(currentTime, timeRange);
|
|
return (float)((relativePositionAtCached(time, timeRange) - timelinePosition) * scrollLength);
|
|
}
|
|
|
|
public double TimeAt(float position, double currentTime, double timeRange, float scrollLength)
|
|
{
|
|
// Convert the position to a length relative to time = 0
|
|
double length = position / scrollLength + relativePositionAt(currentTime, timeRange);
|
|
|
|
// We need to consider all timing points until the specified time and not just the currently-active one,
|
|
// since each timing point individually affects the positions of _all_ hitobjects after its start time
|
|
for (int i = 0; i < controlPoints.Count; i++)
|
|
{
|
|
var current = controlPoints[i];
|
|
var next = i < controlPoints.Count - 1 ? controlPoints[i + 1] : null;
|
|
|
|
// Duration of the current control point
|
|
var currentDuration = (next?.StartTime ?? double.PositiveInfinity) - current.StartTime;
|
|
|
|
// Figure out the length of control point
|
|
var currentLength = currentDuration / timeRange * current.Multiplier;
|
|
|
|
if (currentLength > length)
|
|
{
|
|
// The point is within this control point
|
|
return current.StartTime + length * timeRange / current.Multiplier;
|
|
}
|
|
|
|
length -= currentLength;
|
|
}
|
|
|
|
return 0; // Should never occur
|
|
}
|
|
|
|
private double relativePositionAtCached(double time, double timeRange)
|
|
{
|
|
if (!positionCache.TryGetValue(time, out double existing))
|
|
positionCache[time] = existing = relativePositionAt(time, timeRange);
|
|
return existing;
|
|
}
|
|
|
|
public void Reset() => positionCache.Clear();
|
|
|
|
/// <summary>
|
|
/// Finds the position which corresponds to a point in time.
|
|
/// This is a non-linear operation that depends on all the control points up to and including the one active at the time value.
|
|
/// </summary>
|
|
/// <param name="time">The time to find the position at.</param>
|
|
/// <param name="timeRange">The amount of time visualised by the scrolling area.</param>
|
|
/// <returns>A positive value indicating the position at <paramref name="time"/>.</returns>
|
|
private double relativePositionAt(double time, double timeRange)
|
|
{
|
|
if (controlPoints.Count == 0)
|
|
return time / timeRange;
|
|
|
|
double length = 0;
|
|
|
|
// We need to consider all timing points until the specified time and not just the currently-active one,
|
|
// since each timing point individually affects the positions of _all_ hitobjects after its start time
|
|
for (int i = 0; i < controlPoints.Count; i++)
|
|
{
|
|
var current = controlPoints[i];
|
|
var next = i < controlPoints.Count - 1 ? controlPoints[i + 1] : null;
|
|
|
|
// We don't need to consider any control points beyond the current time, since it will not yet
|
|
// affect any hitobjects
|
|
if (i > 0 && current.StartTime > time)
|
|
continue;
|
|
|
|
// Duration of the current control point
|
|
var currentDuration = (next?.StartTime ?? double.PositiveInfinity) - current.StartTime;
|
|
|
|
// We want to consider the minimal amount of time that this control point has affected,
|
|
// which may be either its duration, or the amount of time that has passed within it
|
|
var durationInCurrent = Math.Min(currentDuration, time - current.StartTime);
|
|
|
|
// Figure out how much of the time range the duration represents, and adjust it by the speed multiplier
|
|
length += durationInCurrent / timeRange * current.Multiplier;
|
|
}
|
|
|
|
return length;
|
|
}
|
|
}
|
|
}
|