Dan Balasescu
bf4be2dfc7
Merge pull request #13782 from peppy/fix-incorrect-filename
...
Fix incorrect filename
2021-07-05 18:53:41 +09:00
smoogipoo
8b7ccdc8b5
Adjust comment
2021-07-05 18:51:23 +09:00
smoogipoo
7d6ab08bb3
Remove unnecessary conditional
2021-07-05 18:49:09 +09:00
smoogipoo
695af31c58
Start alternating at 225BPM as previously
2021-07-05 18:43:27 +09:00
Dean Herbert
7645da7d37
Fix incorrect filename
2021-07-05 18:20:04 +09:00
smoogipoo
2b8efe21ca
Don't ease with 0 time difference
2021-07-05 17:52:10 +09:00
Dean Herbert
229bba14e6
Fix master clock becoming incorrectly paused when all spectator players are too far ahead
2021-07-05 17:45:08 +09:00
smoogipoo
12ca845e55
Use marker class (cleanup)
2021-07-05 17:24:23 +09:00
smoogipoo
6a2c0f772e
Always apply easing, adjust heuristic a bit
2021-07-05 17:22:48 +09:00
smoogipoo
96c0ab8ded
Adjust last frame position when not waiting
2021-07-05 17:11:59 +09:00
Dean Herbert
1e4beddd2d
Disable foreign key enforcing at an sqlite level
2021-07-05 15:41:48 +09:00
Dean Herbert
b7180f16c5
Merge pull request #13778 from Henry-YSLin/fix-unimplemented-mod-test
...
Use a mock ruleset and mod to test unimplemented mods in mod select overlay
2021-07-05 13:54:14 +09:00
Dan Balasescu
8f82ceefe8
Merge pull request #13774 from LumpBloom7/fix-incorrect-first-window
...
Fix nested HitWindows not being considered if parent HitWindow is empty
2021-07-05 13:54:00 +09:00
Dan Balasescu
6b76c54aee
Merge pull request #13756 from PercyDan54/playlist-empty-romanised
...
Fix playlist item displays empty string if no unicode title is present
2021-07-05 13:53:50 +09:00
Dan Balasescu
b82afaddc1
Merge pull request #13767 from peppy/fix-realm-refresh-race
...
Fix thread safety of realm `Refresh` operation
2021-07-05 13:53:29 +09:00
Dean Herbert
d119d9a0db
Merge branch 'master' into adjust-rankings-overlay
2021-07-05 11:59:26 +09:00
Henry Lin
3c37140426
Remove an unused local variable
2021-07-05 09:52:13 +08:00
Henry Lin
bfb603cfeb
Change the test for unimplemented mod to use a mock ruleset and mod
2021-07-05 09:51:24 +08:00
Dean Herbert
f510ef9153
Move previousContext assign within lock to make things look safer
...
Not an actual requirement, but no harm.
2021-07-05 10:49:56 +09:00
Dan Balasescu
b6542a835b
Merge pull request #13757 from peppy/fix-import-optimisation
...
Fix incorrect `base` call causing import optimisation to not work
2021-07-05 10:26:52 +09:00
Dan Balasescu
5f10edee39
Merge branch 'master' into playlist-empty-romanised
2021-07-05 10:20:33 +09:00
Henry Lin
eecf4af029
Rename getSliderBoundingBox and add comments
2021-07-05 09:16:01 +08:00
smoogipoo
cc877f88e2
Fix inspection (create a new ruleset every time)
2021-07-05 10:13:01 +09:00
Derrick Timmermans
216e52d6d0
Avoid using single letter variable names
2021-07-04 23:24:17 +02:00
Derrick Timmermans
1facdcf483
Apply changes to tests
2021-07-04 23:23:24 +02:00
Derrick Timmermans
6d2ffe3a94
Add basic tests
2021-07-04 22:51:35 +02:00
Derrick Timmermans
32b068fbdc
Fix typo causing nested windows to be ignored
2021-07-04 21:50:58 +02:00
Bartłomiej Dach
0fb16cb55f
Merge pull request #13746 from ekrctb/move-caught-object-vertical-offset
...
Move caught object stack vertical offset logic
2021-07-04 18:10:34 +02:00
Derrick Timmermans
0527f669cb
Merge branch 'master' into volume-meter-switch
2021-07-04 17:48:26 +02:00
Bartłomiej Dach
ff56a554f3
Merge branch 'master' into move-caught-object-vertical-offset
2021-07-04 17:26:24 +02:00
Derrick Timmermans
69803105ef
Fix volume meter requesting focus for any action
2021-07-04 17:19:44 +02:00
Bartłomiej Dach
f3cd0be230
Merge pull request #13765 from peppy/fix-initial-volume-display-when-zero
...
Fix volume metre not correctly showing volume when initially zero
2021-07-04 16:55:24 +02:00
PercyDan54
356f5dceef
Add more test case
2021-07-04 22:34:31 +08:00
Bartłomiej Dach
5e148b273e
Merge branch 'master' into fix-initial-volume-display-when-zero
2021-07-04 16:16:01 +02:00
Bartłomiej Dach
a7de3dd81c
Merge pull request #13766 from peppy/fix-zero-hit-score-submit
...
Don't attempt to submit score when nothing has been hit
2021-07-04 16:10:55 +02:00
PercyDan54
44d540eb53
Add test
2021-07-04 22:09:23 +08:00
Derrick Timmermans
14a861003a
Fix code quality errors
2021-07-04 16:06:28 +02:00
Derrick Timmermans
50c9e17e52
Return focus when using UP/DOWN on unfocused meter
2021-07-04 15:42:26 +02:00
Derrick Timmermans
d0707079b1
Remove unused method
2021-07-04 15:35:51 +02:00
Derrick Timmermans
e151c7ffd0
Let VolumeMeter request focus instead of taking it
2021-07-04 15:35:04 +02:00
Bartłomiej Dach
c143a50b73
Merge branch 'master' into fix-zero-hit-score-submit
2021-07-04 15:31:38 +02:00
Bartłomiej Dach
d5fa972215
Merge pull request #13762 from peppy/remove-lazer-references
...
Remove some mentions of "lazer"
2021-07-04 15:01:56 +02:00
Derrick Timmermans
d1553f0864
Implement ability to switch between volume meters
2021-07-04 14:47:07 +02:00
Bartłomiej Dach
19f02dc3d9
Ensure tests with no token have at least one hit
...
Because submission can be prevented by both not having been issued a
correct submission token, and by not actually registering any hits in
gameplay, ensure that tests that don't receive a token register at least
one hit, to avoid potentially having test cases that test the "no token"
flow pass erroneously because they never had any hits in the first
place.
2021-07-04 14:34:52 +02:00
Bartłomiej Dach
3291f15dcc
Mention GetForWrite() as another alternative to Context accesses
2021-07-04 12:08:15 +02:00
Bartłomiej Dach
b89521314f
Mention alternatives to Context when not on update thread in xmldoc
2021-07-04 12:07:34 +02:00
Dean Herbert
3ec7dc3bb9
Update tests in line with thread safety check
2021-07-04 17:59:39 +09:00
Dean Herbert
2a74b1c539
Add test coverage of new scenarios
2021-07-04 17:27:02 +09:00
Dean Herbert
7a710ceffe
Check count as well (statistics can be populated with zero counts)
2021-07-04 16:41:09 +09:00
Dean Herbert
567e9f33a9
Fix thread safety of realm Refresh operation
...
Due to the lack of locking, there was a chance the the update thread
`context` was retrieved just before the `flushContexts` call, followed
by `.Refresh()` being run while the blocking behaviour was invoked.
This can be seen in test failures such as
https://ci.appveyor.com/project/peppy/osu/builds/39859786/tests .
As an aside, I tried multiple different methods to avoid `lock()` on the
update thread but they felt flaky. The overhead of lock when there's no
contention is reportedly around 30-50ns, so likely not of concern. We
can address it at a later point if it becomes one.
2021-07-04 16:24:43 +09:00
