Skip to content

Conversation

@tildejustin
Copy link
Member Author

will try to get around to testing this over the weekend

@tildejustin tildejustin marked this pull request as ready for review November 21, 2024 13:29
@contariaa
Copy link
Member

Atum, StandardSettings and Planifolia look good.
Optifabric also looks fine, although I'm not too familiar with how it's supposed to look the code is mostly written by an outside contributor and as such I think we can consider justins review as the review from the tech advisor team.

I am wondering if there is meant to be a replacement for this deleted inject in SleepBackground, its not a critical issue or anything, but might be an oversight: https://github.com/RedLime/SleepBackground/pull/23/files#diff-48eaa3117ec88f1002ae1ca0401662c6876b5895144a0b3a7b5b590da197a98bL33-L38

Also I remember this being somewhere in javacord but it might be good to have the reasoning for disabling vanillas inactivity fps limiter written down somewhere official because i can no longer find it.
How does vanillas implementation differ from sleepbackgrounds, is there any advantages vanillas implementation has?

contariaa
contariaa previously approved these changes Nov 24, 2024
Copy link
Member

@contariaa contariaa left a comment

Choose a reason for hiding this comment

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

Putting this here from discord so its documented, lgtm!

I am wondering if there is meant to be a replacement for this deleted inject in SleepBackground, its not a critical issue or anything, but might be an oversight: https://github.com/RedLime/SleepBackground/pull/23/files#diff-48eaa3117ec88f1002ae1ca0401662c6876b5895144a0b3a7b5b590da197a98bL33-L38

"in 1.21.2+ it was moved to PieChart, which is rendered in DebugHud, by InGameHud, finally from GameRenderer, which is already canceled via the same LATEST_LOCK_FRAME check so another mixin isn't necessary for that anymore" ~tildejustin

Also I remember this being somewhere in javacord but it might be good to have the reasoning for disabling vanillas inactivity fps limiter written down somewhere official because i can no longer find it.

"seems like fps limiting happens no matter if the player is in the world or not, so my vote would be to just disable its functionality and let sleepbg do what it does best...it hijacks the built in fps limiter to give a custom value under certain conditions"
"customizable, default 1 fps is better than 10"

Knawk
Knawk previously approved these changes Dec 1, 2024
Copy link
Member

@Knawk Knawk left a comment

Choose a reason for hiding this comment

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

The changes since Contaria's approval look good to me.

Knawk
Knawk previously approved these changes Dec 3, 2024
@tildejustin tildejustin merged commit 59dcef6 into Minecraft-Java-Edition-Speedrunning:main Dec 4, 2024
@tildejustin tildejustin added the external mod a change to an external mod label Jan 28, 2025
@tildejustin tildejustin added mod compatibility A mod jar is compatible with another version mod port ports of a legal mod to different versions of minecraft mod update An update to an existing mod and removed mod update An update to an existing mod labels Jan 28, 2025
@tildejustin tildejustin deleted the update/1.21.2-3 branch June 21, 2025 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external mod a change to an external mod mod compatibility A mod jar is compatible with another version mod port ports of a legal mod to different versions of minecraft

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants