Skip to content

chunk optimization#246

Merged
void2012 merged 1 commit intosmartcmd:mainfrom
detectiveren:chunk-render-fix
Mar 3, 2026
Merged

chunk optimization#246
void2012 merged 1 commit intosmartcmd:mainfrom
detectiveren:chunk-render-fix

Conversation

@detectiveren
Copy link
Contributor

Description

Makes chunks load a bit faster, it should be treated as a temp fix for now.

Changes

Increased the amount of chunk rebuilding around the player up to 6 chunks.

Previous Behavior

Chunks would load at a slow pace that in creative mode even the player flying could get ahead of chunk loading

Root Cause

The values for specific variables that influence chunk rendering were low for modern hardware standards.

New Behavior

Chunks should load a bit more faster around the player and the player shouldn't be able to get ahead of chunk rendering.

Fix Implementation

Increased the values for dirty chunk check periods, concurrent chunk rebuilds, chunks that can render within distance from the player and command buffer allocations.

Credit to @twest0001 as they discovered that increasing MAX_COMMANDBUFFER_ALLOCATIONS from 55 * 1024 * 1024 to 2047 * 1024 * 1024 makes chunks load a bit faster too. That is included in this PR along with my changes.

Related Issues

makes chunks load a bit faster
@void2012
Copy link
Collaborator

void2012 commented Mar 3, 2026

If I understand it correctly, the game capped the chunk loader to 110 Megabytes but now it's around 2G?

@codeHusky
Copy link
Collaborator

codeHusky commented Mar 3, 2026

Please pull in main, we've adjusted the Command Allocation thing in a recent commit so your change there is partially redundant. For better compat we should prob be using your way of doing it and revert Xbox One to 512M rather than the higher value it was set to by an earlier PR

@detectiveren
Copy link
Contributor Author

Please pull in main, we've adjusted the Command Allocation thing in a recent commit so your change there is partially redundant. For better compat we should prob be using your way of doing it and revert Xbox One to 512M rather than the higher value it was set to by an earlier PR

Yeah we shouldn't use the Xbox one definition

It should be seperate like 4J intended

@codeHusky
Copy link
Collaborator

If you fully fix up what's in main atm and this tests stable this should be fine to use

@detectiveren
Copy link
Contributor Author

If you fully fix up what's in main atm and this tests stable this should be fine to use

Ah, I hopped off my PC for today just as a finished this PR, my bad lol, would have fixed it instantly

@void2012
Copy link
Collaborator

void2012 commented Mar 3, 2026

Seems ready, no? @codeHusky, @smartcmd

@detectiveren
Copy link
Contributor Author

Seems ready, no? @codeHusky, @smartcmd

yup, I don't see any issues with merging it in tbf

@void2012 void2012 merged commit 1b0e5df into smartcmd:main Mar 3, 2026
EnderActually pushed a commit to EnderActually/MinecraftConsoles that referenced this pull request Mar 3, 2026
makes chunks load a bit faster
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants