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

Skin inconsistency between release and master #24973

Closed
Theighlin opened this issue Sep 29, 2023 · 8 comments · Fixed by #25023
Closed

Skin inconsistency between release and master #24973

Theighlin opened this issue Sep 29, 2023 · 8 comments · Fixed by #25023
Assignees
Labels
area:skinning next release Pull requests which are almost there. We'll aim to get them in the next release, but no guarantees! priority:0 Showstopper. Critical to the next release.

Comments

@Theighlin
Copy link

Theighlin commented Sep 29, 2023

Type

Game behaviour

Bug description

These screenshot were used using the same folder (i renamed osu-development to osu and ran release, not logged in)

Screenshots or videos

2023.924.1 master
image image
image image
image image

Skins: https://osu.ppy.sh/community/forums/topics/1792308?n=1
https://drive.google.com/file/d/1t6-Hfz5mG4IxATLjLcAGmyEEshZob_kp/view?usp=sharing
(i imagine any beatmap will do, but here we go) Beatmap: https://osu.ppy.sh/beatmapsets/1760668#osu/4036420

Version

09c9baa

Logs

performance.log
runtime.log
database.log
input.log

@Theighlin Theighlin changed the title 2 digits hit circle number incorrect on some skins Skin inconsistency between release and master Sep 29, 2023
@frenzibyte
Copy link
Member

This is one of the after-effects of #24706, will investigate further.

@frenzibyte
Copy link
Member

#24975 should fix the incorrect rendering of the glyphs in the first screenshots row. For the hit circles, they look to be over the size of the hit circle's dimensions for the drop-shadow effect:

CleanShot 2023-09-30 at 01 44 26@2x

With #24975, it will render as follows:

CleanShot 2023-09-30 at 01 44 26@2x

We could add bit more leniency to the hit circle's maximum dimensions to allow for such effects, maybe an absolute number applied within WithMaximumSize itself. @peppy thoughts?

@frenzibyte
Copy link
Member

To keep this thread updated, @RockRoller01 has mentioned bringing up some numbers of gameplay elements' dimensions across countless skins in a couple of days, which we can use to alter the current specifications we had set.

@Loreos7
Copy link
Contributor

Loreos7 commented Sep 30, 2023

Release Master
image image
image image
image image

Skins: BTMC Freedom Dive, Bacon boi 1.0 blue

@peppy peppy added priority:0 Showstopper. Critical to the next release. next release Pull requests which are almost there. We'll aim to get them in the next release, but no guarantees! labels Oct 3, 2023
@peppy
Copy link
Member

peppy commented Oct 3, 2023

Of note, this needs to be resolved before the next release. Increasing by the amount required to support the skin above would be a good start if we don't hear back from rockroller or find the time to investigate further ourselves.

@RockRoller01
Copy link

I've gone and grabbed stats for all the files that @bdach listed on discord. For animated files I checked for their base version and the first frame, checking all frames didnt really feel worth it. Overall this goes over ~4200 skins.

https://gist.github.com/RockRoller01/0a25249c86fda4c5a0c3d8fb97c28041

If stats for any other files are needed just let me know, now that I have working code it takes no time to get them.

@Loreos7
Copy link
Contributor

Loreos7 commented Oct 5, 2023

I can still see the small difference. There are harsh shadows appear near a note.

Release Master with #24975
image image
image image

@peppy
Copy link
Member

peppy commented Oct 5, 2023

I can still see the small difference. There are harsh shadows appear near a note.

If you're talking about master after #24975, then this is to be expected. Further changes still need to be applied in order to fix this issue, which is why it's still open.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:skinning next release Pull requests which are almost there. We'll aim to get them in the next release, but no guarantees! priority:0 Showstopper. Critical to the next release.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants