Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rewrite judgement articles #8245

Merged
merged 24 commits into from
Jul 2, 2023
Merged

Rewrite judgement articles #8245

merged 24 commits into from
Jul 2, 2023

Conversation

Walavouchey
Copy link
Member

@Walavouchey Walavouchey commented Oct 22, 2022

part of #8189

stuff to check

  • Opinions on whether we want to use the new judgement names throughout articles. I thought it seemed ok, even though technically they only appear in lazer.
    • Blocked on finalised judgement names? another day
  • The mania Perfect, Meh, and Miss windows differ from the OD article as i've taken all of them from lazer. Will need to confirm whether this is an actual gameplay difference between lazer and stable.
    • Turns out the OD article was just wrong and put the Miss window in the place of the Meh window (and left out miss windows)
  • Confirm hit window/hit error rounding shenanigans
    • The hit windows are truncated after accounting for OD, while the hit error uses rounded time derived from the audio engine.
    • DT and HT make the clock tick faster and slower respectively, which scales the hit error and hit windows, which may affect how much the hit windows can vary. Haven't thought it through thoroughly.
  • Confirm spinner spin requirements

remarks

  • The mania hold note table was created by empirical testing (I have no idea what the logic there is, but it's different from lazer I believe)
  • no idea what to do about the catch article, it doesn't seem to have the concept of "judgements" in the same way anyway

future effort

  • separate article for judgement names in other rhythm games
  • fix mentions of judgements across the wiki
  • move slightly out-of-scope stuff like score and accuracy-related information out of the judgement articles
  • fill index page with a more general gameplay-related description about judgements (judgement line, approach circle)

wiki/Gameplay/Judgement/osu!/en.md Outdated Show resolved Hide resolved
wiki/Gameplay/Judgement/osu!/en.md Outdated Show resolved Hide resolved
wiki/Gameplay/Judgement/osu!/en.md Outdated Show resolved Hide resolved
wiki/Gameplay/Judgement/osu!/en.md Outdated Show resolved Hide resolved
wiki/Gameplay/Judgement/osu!/en.md Show resolved Hide resolved
wiki/Gameplay/Judgement/osu!/en.md Outdated Show resolved Hide resolved
wiki/Gameplay/Judgement/osu!/en.md Outdated Show resolved Hide resolved
wiki/Gameplay/Judgement/osu!/en.md Outdated Show resolved Hide resolved
wiki/Gameplay/Judgement/osu!catch/en.md Outdated Show resolved Hide resolved
Comment on lines -15 to -20
| ![](img/catch-300.png "Fruit") | Fruit | A large fruit which is equivalent to normal Hit-circle or Slider's starting, repeating and ending point. Catching it is equivalent to perfect hit (shown as 300 on multiplayer and result). |
| ![](img/catch-100.png "Juice Drop") | Juice Drops (Slider Tick) | These large droplets signifies Slider's ticks, similar to the dots (slider ticks) inside the sliders. It is compulsory to catch for combo. It is twice the size of Droplet (Slider Trails). 30pt for successful capture (shown as 100 on multiplayer and result). |
| ![](img/catch-50.png "Droplet") | Droplets (Slider Trail) | These small droplets are slider trails. They form the trails of the slider and are smaller comparatively to Juice Drops. 10pt for each Droplets. It would decrease accuracy for not capturing but does not break combo count (shown as 50 on multiplayer and result) |
| (None) | Miss Droplets | Droplets (Slider Trails) not caught. Not shown in Results screen but is shown in server-side. Combo will not be broken but will decrease accuracy and lose out one of the 50's score. Not considered a miss. Not shown at multiplayer and result but shown as *Droplet Miss* in website leaderboard. |
| ![](img/catch-banana.png "Banana") | Bananas | Spinner's equivalent. These pesky Bananas will become smaller as they fall towards the catcher. It is not compulsory to catch them. 1,100pt given, however this one follows spinner clear in osu! which combines 1000 as bonus and 100 as spinning score. |
| ![](img/catch-0.png "Miss") | Miss | Normal Fruit and Juice Drops not caught. Breaks combo count. Given when failed to catch the Fruit/Juice Drops. No score given and breaks combo. This can only be seen from the results screen. |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

might be extra material for respective hit object articles

@TicClick
Copy link
Contributor

random suggestion: mention o!m's judgement line maybe? or do you think it deserves its own article, like osu!'s approach circle?

@Walavouchey
Copy link
Member Author

random suggestion: mention o!m's judgement line maybe? or do you think it deserves its own article, like osu!'s approach circle?

hmm i have an idea on how this should work but i won't have time to address it for at least two weeks

i.e. not in this pr

@Walavouchey
Copy link
Member Author

pinging @peppy / @smoogipoo for the "stuff to check" checklist

@peppy
Copy link
Sponsor Member

peppy commented Jan 11, 2023

Opinions on whether we want to use the new judgement names throughout articles. I thought it seemed ok, even though technically they only appear in lazer.

We are all in agreement on doing this. That said, we're also considering renaming the internal terms to avoid upsetting a portion of the community, so it would be worth waiting for that to happen and be finalised (likely will be done this month).

Previous: Miss < Meh < OK < Good < Great < Perfect
Proposed: Miss < Bad < OK < Good < Great < MAX

The mania Perfect, Meh, and Miss windows differ from the OD article as i've taken all of them from lazer. Will need to confirm whether this is an actual gameplay difference between lazer and stable.

I've given you access to the stable reference code so you can hopefully check this. Let me know if you can't figure it out and I'll go look for you.

Confirm hit window/hit error rounding shenanigans

I presume this is the off-by-0.5ms thing? I think there's still an open discussion on that potentially being different in lazer, so would be best to track that there. See ppy/osu#11311.

Confirm spinner spin requirements

Ha ha. I'd highly recommend not attempting to figure this out from the stable code base. I think there's a group of community members that figured out a formula for this, but I haven't confirmed its accuracy yet. I'll try and find a link to this, but you may find it faster than I can (was in a reddit thread related to spinning).

@Walavouchey
Copy link
Member Author

I think there's a group of community members that figured out a formula for this

I do know about this as i've learned from a certain reddit post about spinners. I have considered contacting them, yes

Copy link
Member Author

@Walavouchey Walavouchey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would it be good to add internal reference comments, linking to the stable reference repo?

Comment on lines 71 to 72
| Minimum spins per second | `1.5 + 0.2 * OD` if OD < 5, `1.25 + 0.25 * OD` if OD >= 5 |
| Minimum spins required | Spinner length in seconds * minimum spins per second + 0.5 |
Copy link
Member Author

@Walavouchey Walavouchey Jan 28, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

after checking with stable source apparently there were some random bits of information that were inconsistent or not mentioned

  • the formula for the minimum required spins per second from https://osu.ppy.sh/wiki/en/Beatmap/Overall_difficulty#sliders-and-spinners is the exact formula used in the code, except that it deals in half rotations which has been adjusted here
  • this random +0.5 relates to the "All spinners required half of a spin more to complete." trivia in the history section. all spinners really were nerfed by 1/2 rotation for some ungodly reason (before it would've been +1)
    • due to this the "minimum spins per second" formula listed here is actually slightly less than what you'd need (by 0.5 / length in seconds so not much in actuality)
  • the "clear" text for spinners actually appears half of a rotation before you'd get a GREAT, and half of a rotation after you'd get an OK (if you stopped immediately you'd get an OK)
  • the "One spin less than the required number" for the OK requirement was in terms of full spins, so that part didn't need changing

I'd highly recommend not attempting to figure this out from the stable code base.

well the judgement part wasn't so bad to investigate, but figuring out maximum score possible (involving maximum possible rotations) is way more involved. there is a desmos calculator for this though and it's true that it would be better to empirically test that against stable (but that info would be put somewhere like Gameplay/Score or Gameplay/Hit_object/Spinner)

@TicClick
Copy link
Contributor

would it be good to add internal reference comments, linking to the stable reference repo?

I think so. would use inline HTML comments, given that the repository is virtually inaccessible by anyone other than a handful of people (if it was more widespread, but still vastly limited, that would've been worth putting in the footnotes)

the one initially written for mania only applies for <=, while < works a
bit differently
@Walavouchey
Copy link
Member Author

  • Blocked on finalised judgement names?

at this point i don't think this pr should be blocked on something silly like this. i'm more inclined to just go with these changes as is and use the current judgement names for now, so i can move on with the rest of the planned changes.

wiki/Gameplay/Judgement/osu!/en.md Outdated Show resolved Hide resolved
wiki/Gameplay/Judgement/osu!mania/en.md Outdated Show resolved Hide resolved
wiki/Gameplay/Judgement/osu!mania/en.md Outdated Show resolved Hide resolved
wiki/Gameplay/Judgement/osu!taiko/en.md Outdated Show resolved Hide resolved
wiki/Gameplay/Judgement/osu!taiko/en.md Outdated Show resolved Hide resolved
wiki/Gameplay/Judgement/osu!taiko/en.md Outdated Show resolved Hide resolved
wiki/Gameplay/Judgement/osu!taiko/en.md Outdated Show resolved Hide resolved
wiki/Gameplay/Judgement/osu!taiko/en.md Outdated Show resolved Hide resolved
@TicClick TicClick merged commit 43aa27f into ppy:master Jul 2, 2023
2 checks passed
@Walavouchey Walavouchey deleted the judgement branch July 2, 2023 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants