mirror of https://github.com/ppy/osu
Move object counts to BeatmapInfo
This commit is contained in:
parent
fdcf87569c
commit
0171078052
|
@ -21,6 +21,10 @@ public class TournamentBeatmap : IBeatmapInfo, IBeatmapSetOnlineInfo
|
|||
|
||||
public double StarRating { get; set; }
|
||||
|
||||
public int EndTimeObjectCount { get; set; }
|
||||
|
||||
public int TotalObjectCount { get; set; }
|
||||
|
||||
public IBeatmapMetadataInfo Metadata { get; set; } = new BeatmapMetadata();
|
||||
|
||||
public IBeatmapDifficultyInfo Difficulty { get; set; } = new BeatmapDifficulty();
|
||||
|
@ -41,6 +45,8 @@ public TournamentBeatmap(APIBeatmap beatmap)
|
|||
Metadata = beatmap.Metadata;
|
||||
Difficulty = beatmap.Difficulty;
|
||||
Covers = beatmap.BeatmapSet?.Covers ?? new BeatmapSetOnlineCovers();
|
||||
EndTimeObjectCount = beatmap.EndTimeObjectCount;
|
||||
TotalObjectCount = beatmap.TotalObjectCount;
|
||||
}
|
||||
|
||||
public bool Equals(IBeatmapInfo? other) => other is TournamentBeatmap b && this.MatchesOnlineID(b);
|
||||
|
|
|
@ -20,7 +20,6 @@
|
|||
using osu.Game.Scoring;
|
||||
using osu.Game.Scoring.Legacy;
|
||||
using osu.Game.Screens.Play;
|
||||
using Realms;
|
||||
|
||||
namespace osu.Game
|
||||
{
|
||||
|
@ -177,9 +176,13 @@ private void processBeatmapsWithMissingObjectCounts()
|
|||
{
|
||||
Logger.Log("Querying for beatmaps with missing hitobject counts to reprocess...");
|
||||
|
||||
HashSet<Guid> beatmapIds = realmAccess.Run(r => new HashSet<Guid>(r.All<BeatmapInfo>()
|
||||
.Filter($"{nameof(BeatmapInfo.Difficulty)}.{nameof(BeatmapDifficulty.TotalObjectCount)} == 0")
|
||||
.AsEnumerable().Select(b => b.ID)));
|
||||
HashSet<Guid> beatmapIds = new HashSet<Guid>();
|
||||
|
||||
realmAccess.Run(r =>
|
||||
{
|
||||
foreach (var b in r.All<BeatmapInfo>().Where(b => b.TotalObjectCount == 0))
|
||||
beatmapIds.Add(b.ID);
|
||||
});
|
||||
|
||||
Logger.Log($"Found {beatmapIds.Count} beatmaps which require reprocessing.");
|
||||
|
||||
|
|
|
@ -21,9 +21,6 @@ public class BeatmapDifficulty : EmbeddedObject, IBeatmapDifficultyInfo
|
|||
public double SliderMultiplier { get; set; } = 1.4;
|
||||
public double SliderTickRate { get; set; } = 1;
|
||||
|
||||
public int EndTimeObjectCount { get; set; }
|
||||
public int TotalObjectCount { get; set; }
|
||||
|
||||
public BeatmapDifficulty()
|
||||
{
|
||||
}
|
||||
|
@ -47,9 +44,6 @@ public virtual void CopyTo(BeatmapDifficulty difficulty)
|
|||
|
||||
difficulty.SliderMultiplier = SliderMultiplier;
|
||||
difficulty.SliderTickRate = SliderTickRate;
|
||||
|
||||
difficulty.EndTimeObjectCount = EndTimeObjectCount;
|
||||
difficulty.TotalObjectCount = TotalObjectCount;
|
||||
}
|
||||
|
||||
public virtual void CopyFrom(IBeatmapDifficultyInfo other)
|
||||
|
@ -61,9 +55,6 @@ public virtual void CopyFrom(IBeatmapDifficultyInfo other)
|
|||
|
||||
SliderMultiplier = other.SliderMultiplier;
|
||||
SliderTickRate = other.SliderTickRate;
|
||||
|
||||
EndTimeObjectCount = other.EndTimeObjectCount;
|
||||
TotalObjectCount = other.TotalObjectCount;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
|
@ -388,9 +388,7 @@ private List<BeatmapInfo> createBeatmapDifficulties(BeatmapSetInfo beatmapSet, R
|
|||
OverallDifficulty = decodedDifficulty.OverallDifficulty,
|
||||
ApproachRate = decodedDifficulty.ApproachRate,
|
||||
SliderMultiplier = decodedDifficulty.SliderMultiplier,
|
||||
SliderTickRate = decodedDifficulty.SliderTickRate,
|
||||
EndTimeObjectCount = decoded.HitObjects.Count(h => h is IHasDuration),
|
||||
TotalObjectCount = decoded.HitObjects.Count
|
||||
SliderTickRate = decodedDifficulty.SliderTickRate
|
||||
};
|
||||
|
||||
var metadata = new BeatmapMetadata
|
||||
|
@ -428,6 +426,8 @@ private List<BeatmapInfo> createBeatmapDifficulties(BeatmapSetInfo beatmapSet, R
|
|||
GridSize = decodedInfo.GridSize,
|
||||
TimelineZoom = decodedInfo.TimelineZoom,
|
||||
MD5Hash = memoryStream.ComputeMD5Hash(),
|
||||
EndTimeObjectCount = decoded.HitObjects.Count(h => h is IHasDuration),
|
||||
TotalObjectCount = decoded.HitObjects.Count
|
||||
};
|
||||
|
||||
beatmaps.Add(beatmap);
|
||||
|
|
|
@ -120,6 +120,10 @@ public BeatmapOnlineStatus Status
|
|||
[JsonIgnore]
|
||||
public bool Hidden { get; set; }
|
||||
|
||||
public int EndTimeObjectCount { get; set; }
|
||||
|
||||
public int TotalObjectCount { get; set; }
|
||||
|
||||
/// <summary>
|
||||
/// Reset any fetched online linking information (and history).
|
||||
/// </summary>
|
||||
|
|
|
@ -91,8 +91,8 @@ public void ProcessObjectCounts(BeatmapInfo beatmapInfo, MetadataLookupScope loo
|
|||
var working = workingBeatmapCache.GetWorkingBeatmap(beatmapInfo);
|
||||
var beatmap = working.Beatmap;
|
||||
|
||||
beatmapInfo.Difficulty.EndTimeObjectCount = beatmap.HitObjects.Count(h => h is IHasDuration);
|
||||
beatmapInfo.Difficulty.TotalObjectCount = beatmap.HitObjects.Count;
|
||||
beatmapInfo.EndTimeObjectCount = beatmap.HitObjects.Count(h => h is IHasDuration);
|
||||
beatmapInfo.TotalObjectCount = beatmap.HitObjects.Count;
|
||||
|
||||
// And invalidate again afterwards as re-fetching the most up-to-date database metadata will be required.
|
||||
workingBeatmapCache.Invalidate(beatmapInfo);
|
||||
|
|
|
@ -44,19 +44,6 @@ public interface IBeatmapDifficultyInfo
|
|||
/// </summary>
|
||||
double SliderTickRate { get; }
|
||||
|
||||
/// <summary>
|
||||
/// The number of hitobjects in the beatmap with a distinct end time.
|
||||
/// </summary>
|
||||
/// <remarks>
|
||||
/// Canonically, these are hitobjects are either sliders or spinners.
|
||||
/// </remarks>
|
||||
int EndTimeObjectCount { get; }
|
||||
|
||||
/// <summary>
|
||||
/// The total number of hitobjects in the beatmap.
|
||||
/// </summary>
|
||||
int TotalObjectCount { get; }
|
||||
|
||||
/// <summary>
|
||||
/// Maps a difficulty value [0, 10] to a two-piece linear range of values.
|
||||
/// </summary>
|
||||
|
|
|
@ -61,5 +61,18 @@ public interface IBeatmapInfo : IHasOnlineID<int>, IEquatable<IBeatmapInfo>
|
|||
/// The basic star rating for this beatmap (with no mods applied).
|
||||
/// </summary>
|
||||
double StarRating { get; }
|
||||
|
||||
/// <summary>
|
||||
/// The number of hitobjects in the beatmap with a distinct end time.
|
||||
/// </summary>
|
||||
/// <remarks>
|
||||
/// Canonically, these are hitobjects are either sliders or spinners.
|
||||
/// </remarks>
|
||||
int EndTimeObjectCount { get; }
|
||||
|
||||
/// <summary>
|
||||
/// The total number of hitobjects in the beatmap.
|
||||
/// </summary>
|
||||
int TotalObjectCount { get; }
|
||||
}
|
||||
}
|
||||
|
|
|
@ -41,6 +41,10 @@ public class APIBeatmap : IBeatmapInfo, IBeatmapOnlineInfo
|
|||
[JsonProperty(@"difficulty_rating")]
|
||||
public double StarRating { get; set; }
|
||||
|
||||
public int EndTimeObjectCount => SliderCount + SpinnerCount;
|
||||
|
||||
public int TotalObjectCount => CircleCount + SliderCount + SpinnerCount;
|
||||
|
||||
[JsonProperty(@"drain")]
|
||||
public float DrainRate { get; set; }
|
||||
|
||||
|
@ -108,9 +112,7 @@ private double hitLengthInSeconds
|
|||
DrainRate = DrainRate,
|
||||
CircleSize = CircleSize,
|
||||
ApproachRate = ApproachRate,
|
||||
OverallDifficulty = OverallDifficulty,
|
||||
EndTimeObjectCount = SliderCount + SpinnerCount,
|
||||
TotalObjectCount = CircleCount + SliderCount + SpinnerCount
|
||||
OverallDifficulty = OverallDifficulty
|
||||
};
|
||||
|
||||
IBeatmapSetInfo? IBeatmapInfo.BeatmapSet => BeatmapSet;
|
||||
|
|
|
@ -62,8 +62,8 @@ public class LegacyBeatmapConversionDifficultyInfo : IBeatmapDifficultyInfo
|
|||
SourceRuleset = beatmapInfo.Ruleset,
|
||||
CircleSize = beatmapInfo.Difficulty.CircleSize,
|
||||
OverallDifficulty = beatmapInfo.Difficulty.OverallDifficulty,
|
||||
EndTimeObjectCount = beatmapInfo.Difficulty.EndTimeObjectCount,
|
||||
TotalObjectCount = beatmapInfo.Difficulty.TotalObjectCount
|
||||
EndTimeObjectCount = beatmapInfo.EndTimeObjectCount,
|
||||
TotalObjectCount = beatmapInfo.TotalObjectCount
|
||||
};
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue