Commit Graph

30154 Commits

Author SHA1 Message Date
eb04cc9213 Update framework 2021-12-27 23:04:10 +09:00
5a591713ae Fix legacy combo counter not fully hiding for rulesets that implement their own 2021-12-26 16:49:20 +01:00
724b601d2a Rename AllowSampleLookup argument to something saner 2021-12-26 14:29:07 +01:00
dc21c8901e Always allow looking up storyboard samples from beatmap skin 2021-12-26 14:25:28 +01:00
8ad53138a6 Merge pull request #16234 from smoogipoo/fix-initial-button-colour
Fix OsuAnimatedButton animating when initially disabled
2021-12-25 15:34:40 +09:00
957a1648c7 Merge pull request #16237 from smoogipoo/fix-finished-play-exception-2
Improve reliability of exiting gameplay in multiplayer
2021-12-25 15:32:14 +09:00
ac1a452e20 Merge pull request #16239 from peppy/sanitise-test-directory-usage
Fix some more test directory usage weirdness
2021-12-25 12:09:18 +09:00
7088471cad Protect against test failures due to logger being set too early (and beginning logging) 2021-12-25 02:21:49 +09:00
a9a7a33974 Update framework 2021-12-25 02:04:47 +09:00
a43cc20ae2 Apply changes to GameplayCompleted from reviews 2021-12-24 21:58:20 +09:00
c6854b37c8 Unbind event on disposal 2021-12-24 21:58:09 +09:00
d36fe3af09 Merge pull request #16238 from peppy/fix-incorrect-delegate-capture
Fix incorrect delegate capture leading to slow leak of audio tracks
2021-12-24 21:34:04 +09:00
1c0dc204c4 Fix some regressing behaviour due to CleanRunHeadlessGameHost's naming structure changing 2021-12-24 20:24:37 +09:00
8cdad5f4d9 Remove incorrectly used gameSuffix from CleanRunHeadlessGameHost 2021-12-24 20:15:10 +09:00
2ee3e61983 Merge branch 'master' into beatmap-card/extra-on-listing 2021-12-24 19:02:51 +09:00
cc7089c3f4 Cancel more liberally 2021-12-24 19:00:09 +09:00
d602aebebb Add missing cancellation token and rename load task variable to match purpose 2021-12-24 18:58:31 +09:00
842d508aee Fix incorrect delgate capture leading to slow leak of audio tracks
During profile, it was found that the `Completed` delegate was
incorrectly also capturing `lastTrack`, leading to an unexpected
reference chain that led to a memory leak over a long period of time.

This solves the issue by moving the delegate construction to its own
method, where it won't capture the other variables.
2021-12-24 18:39:29 +09:00
83a1d39f80 Merge pull request #16231 from bdach/ruleset-config-cache-overwriting-3
Fix ruleset config cache potentially not working correctly in test contexts
2021-12-24 17:18:42 +09:00
3ca238825b Unify width of normal and extra card sizes 2021-12-24 09:17:07 +01:00
18c144395a Rename to dimColour 2021-12-24 17:03:29 +09:00
f44af1ad69 Move bind to LoadComplete 2021-12-24 17:00:59 +09:00
727335dcad Improve reliability of exiting gameplay 2021-12-24 14:23:09 +09:00
c973739b08 Fix OsuButton event binding + inital-enablement colour 2021-12-24 11:56:12 +09:00
c8073000c5 Fix initally-disabled OsuAnimatedButton colour fading 2021-12-24 11:56:12 +09:00
ce4389576f Merge branch 'master' into fix-incorrect-reference 2021-12-24 10:03:11 +09:00
c429c74d89 Expose and use test ruleset config cache in test scenes 2021-12-23 19:42:26 +01:00
8094b502cb Remove test-specific logic from RulesetConfigCache 2021-12-23 19:42:26 +01:00
77da1e12d5 Add test implementation of the ruleset config cache 2021-12-23 19:40:55 +01:00
5cbaa028eb Use extracted ruleset config cache implementation in DI 2021-12-23 19:02:10 +01:00
0f65c4e960 Extract interface for ruleset config cache 2021-12-23 18:59:02 +01:00
a49a4329ee Add capability to switch between card sizes 2021-12-23 15:55:37 +01:00
d0427ec85f Add support card size tab control to beatmap listing 2021-12-23 15:55:37 +01:00
1876617d8e Implement beatmap card size tab control 2021-12-23 15:55:36 +01:00
cd4c1bc678 Add factory method for various card sizes 2021-12-23 15:55:36 +01:00
998b07db90 Fix TestMultiplayerClient referencing the wrong room 2021-12-23 19:14:09 +09:00
1040590844 Add cancellation support to game-side IResourceStores 2021-12-23 10:33:17 +01:00
0732a9e6da Update framework 2021-12-23 18:08:44 +09:00
d69552fbcf Fix potential crash from cross-thread drawable mutation in collection management
As seen at https://github.com/peppy/osu/runs/4613800663?check_suite_focus=true.
2021-12-23 13:21:17 +09:00
e1b539fa9e Merge pull request #16213 from peppy/working-beatmap-live
Tidy up `WorkingBeatmap`
2021-12-23 13:02:37 +09:00
25e38560ce Fix placeholder drawables on beatmap listing not always hiding correctly
`BeatmapListingOverlay.addContentToPlaceholder()`, in order to make
transitions between different beatmap listing content (whether it is
actual cards, or placeholders for no beatmaps found/supporter-specific
filters chosen), would set `BypassAutoSizeAxes = Y` on content as it is
fading out, to make the transition smoother. The property in question
was supposed to be getting restored to `None` on the next show.

In testing scenarios, it sometimes turned out that this wasn't the case,
therefore making the placeholders effectively not show - while they
were present and fully opaque, they would be the only child of
an auto-sized container with `BypassAutoSizeAxes = Y`, so the parent
auto-sized to a zero height, which logically follows from the premise,
but is not what was desired.

This in turn was caused by the fact that the `BypassAutoSizeAxes = Y`
set was scheduled, and sometimes it would be scheduled in such a way
that the drawable would cease to be present on the next frame due to its
alpha being past the cutoff point of 0.0001. Therefore the scheduled set
would not execute until the *next* time the placeholder was shown,
therefore causing the bug.

Fix by ensuring that the placeholder drawables are always present if
their schedulers have any tasks enqueued, on top of the usual checks of
alpha and scale performed via the base implementation.
2021-12-22 15:05:23 +01:00
f179ff33c9 Remove all NRT changes
This reverts commit bb9877adab.
2021-12-22 21:02:06 +09:00
bb9877adab Reapply IWorkingBeatmap nullability fixed 2021-12-22 20:48:48 +09:00
67e6b50c12 Merge pull request #16135 from stanriders/refactor-diffcalc
Slightly refactor difficulty and pp calculators
2021-12-22 19:44:02 +09:00
fa5ee5f72c Merge pull request #16212 from peppy/fix-beatmap-leaderboard-global-reference
Fix potential errors thrown during beatmap leaderboard display due to incorrect beatmap reference
2021-12-22 19:26:32 +09:00
bc47801cbb Revert "Apply NRT to WorkingBeatmap"
This reverts commit 9e17d7d4e3.
2021-12-22 19:14:18 +09:00
9e17d7d4e3 Apply NRT to WorkingBeatmap 2021-12-22 18:41:45 +09:00
53c0a6708f Move public accessor properties to top of file 2021-12-22 18:25:09 +09:00
a5f77a86ef Move load methods and rename loadedTrack to match other private fields 2021-12-22 18:24:51 +09:00
7344c87b16 Remove RecyclableLazy 2021-12-22 18:18:12 +09:00