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

Add suboption to Hidden (HD) for decreasing visibility based on combo #31087

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

DavidBeh
Copy link
Contributor

@DavidBeh DavidBeh commented Dec 11, 2024

Video:

https://www.youtube.com/watch?v=Hc0kffA_Ewk

This implementation addresses #30871 for osu!std.

When a player's combo breaks, all approach circles on the screen become visible. As the player's combo increases again, the visibility of new approach circles gradually decreases until they disappear completely once the combo reaches the value specified by the player. The mod tracks and uses an internal combo value that is influenced solely by the results of HitCircles and SliderHeads.

This behavior can also be easily adapted for hidden mods in other rulesets. ModHidden handles most of the necessary logic and provides extensions for implementers to adjust key aspects, after which they only need to apply the opacity provided by ModHidden.

@bdach
Copy link
Collaborator

bdach commented Dec 12, 2024

I dunno if we want this in the game. Leaning no by default unless there is actual support.

@DavidBeh
Copy link
Contributor Author

Thanks for taking a look.

Leaning no by default unless there is actual support.

Support from whom? Apart from your input, I’ve only seen neutral or positive reactions in both the GitHub Discussion and the Development Discord Server. Since the option is disabled by default, it would be unintrusive to players.

Additionally, the concept of adjusting a mod's difficulty based on player performance is already well-established in the game. Even some ranked mods, like Blinds, have similar mechanics. (Also, I don’t necessarily want this option to be ranked.)

Personally, I believe this feature offers a great way to ease into playing the hidden mod or practicing beatmaps with it enabled.

Are there specific concerns you have with the concept, the behavior of the implementation, or the code itself?

Also, I am aware of the failing CI tests. I will look into these in the following two weeks.

@bdach
Copy link
Collaborator

bdach commented Dec 12, 2024

Are there specific concerns you have with the concept, the behavior of the implementation, or the code itself?

I have only one concern and it's very simple: it's extra complexity and I don't want extra complexity if there isn't a very good reason for it.

What you got in terms of support is 2 upvotes (not counting yourself) on the discord discussion. I'm not sure what happened on discord because it's an ephemeral communication medium in my mind and I don't 100% keep up with it.

I would want to see a considerably higher number of supporters in order to consider adding this feature.

@DavidBeh
Copy link
Contributor Author

DavidBeh commented Dec 21, 2024

Is this enough support?

@bdach
Copy link
Collaborator

bdach commented Dec 23, 2024

Results look 50-50ish. I'm not particularly positively swayed by that result, so I guess not.

@DavidBeh
Copy link
Contributor Author

Results look 50-50ish. I'm not particularly positively swayed by that result

I only added negative voting options for statistical/transparancy purposes and I don't think that the ratio should serve as an argument for weither the feature should be implemented.

The reason is, that the feature is disabled by default and the only other effect is a second control for this mods preferences-menu. Nobody can seriously be disrupted by this. So the only reasons for people to vote against are:

  • they would not use it, because don't need it/no lazer/no std/etc...
  • they only read the title (or other misunderstandings)
  • adds too much complexity

Apart from the last one, these are not valid arguments adding this feature. And as for complexity, almost every addition adds complexity. And I think in this case, where it's just in a corner of the codebase and well documented by me, this argument looks weak.

In conclusion, counting all sources there have already been more than 10 people expressing support for this (i admit partially because of me promoting it on various channels), which is the only useful metric we got, so why can't we move forward with this?

@ominoussage
Copy link

ominoussage commented Dec 24, 2024

Looks cool but having some circles not hidden shouldn't have the HD score multiplier bonus and the score bonus should also adjust based on how Hidden the circles are. Otherwise, this will give players an advantage when they combo break and that will be unfair for all the old HD scores that also combo broke.

It's kinda complicated but if you manage to do that, then this'll make Hidden playable for everybody. I'm fine with this being added if you address my concerns.

@DavidBeh
Copy link
Contributor Author

DavidBeh commented Dec 24, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants