mirror of
https://github.com/ppy/osu
synced 2024-12-26 08:53:10 +00:00
b0cfea4916
Standardised score conversion would return a negative total score for https://osu.ppy.sh/scores/taiko/182230346. The underlying reason for this is that the estimation of the accuracy portion for a score can be above the actual accuracy portion in the taiko ruleset. When calculating the maximum accuracy portion achievable, `TaikoLegacyScoreSimulator` will include the extra 300 points from a double hit on a strong hit, per strong hit. However, this double hit is not factored into accuracy. Both of the aforementioned facts mean that in taiko maximumLegacyAccuracyScore * score.Accuracy - which normally in other rulesets can be used pretty reliably as the exact number of points gained from the accuracy portion - is an estimate in the case of taiko, and an _upper_ estimate at that, because it implicitly assumes that the user has also hit `score.Accuracy` percent of all double hits possible in the beatmap. If this assumption is not upheld, then the user will have earned _less_ points than that from the accuracy portion, which means that the combo proportion estimate will go below zero. It is possible that this has happened on other scores before, but did not result in the total score going negative as the accuracy portion gained would have counteracted the effect of that due to being larger in magnitude than the score loss incurred from the negative combo portion. In the case of the score in question this was not the case due to very low accuracy _and_ very low max combo. |
||
|---|---|---|
| .. | ||
| BeatmapExporter.cs | ||
| BeatmapLookupCache.cs | ||
| EmptyRealmSet.cs | ||
| ICanAcceptFiles.cs | ||
| IHasFiles.cs | ||
| IHasGuidPrimaryKey.cs | ||
| IHasNamedFiles.cs | ||
| IHasOnlineID.cs | ||
| IHasPrimaryKey.cs | ||
| IHasRealmFiles.cs | ||
| IModelDownloader.cs | ||
| IModelFileManager.cs | ||
| IModelImporter.cs | ||
| IModelManager.cs | ||
| ImportParameters.cs | ||
| ImportProgressNotification.cs | ||
| ImportTask.cs | ||
| INamedFile.cs | ||
| INamedFileInfo.cs | ||
| INamedFileUsage.cs | ||
| IPostNotifications.cs | ||
| ISoftDelete.cs | ||
| LegacyArchiveExporter.cs | ||
| LegacyBeatmapExporter.cs | ||
| LegacyBeatmapImporter.cs | ||
| LegacyCollectionImporter.cs | ||
| LegacyExporter.cs | ||
| LegacyImportManager.cs | ||
| LegacyModelImporter.cs | ||
| LegacyScoreExporter.cs | ||
| LegacyScoreImporter.cs | ||
| LegacySkinExporter.cs | ||
| LegacySkinImporter.cs | ||
| Live.cs | ||
| MemoryCachingComponent.cs | ||
| MissingBeatmapNotification.cs | ||
| ModelDownloader.cs | ||
| ModelManager.cs | ||
| OnlineLookupCache.cs | ||
| RealmAccess.cs | ||
| RealmArchiveModelImporter.cs | ||
| RealmExtensions.cs | ||
| RealmFileStore.cs | ||
| RealmLive.cs | ||
| RealmLiveUnmanaged.cs | ||
| RealmObjectExtensions.cs | ||
| StandardisedScoreMigrationTools.cs | ||
| TooManyDownloadsNotification.cs | ||
| UserLookupCache.cs | ||