Skip to content

Load the vanilla world height of columns forced through ForgeChunkManager - #158

Merged
RecursivePineapple merged 1 commit into
GTNewHorizons:masterfrom
JakfutCC:fix-column-tickets
Oct 31, 2026
Merged

RecursivePineapple merged 1 commit into
GTNewHorizons:masterfrom
JakfutCC:fix-column-tickets

Conversation

@Jakfut

@Jakfut Jakfut commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Summary

A column loaded with ForgeChunkManager.forceChunk(ticket, ChunkCoordIntPair) loaded no cubes, CubicChunkManager.onForgeChunkManagerForceChunk was an empty TODO. Chunk loaders written for vanilla (server utils claims, OpenComputers, ...) only worked by accident. Vanilla's per tick checks of the forced column loaded its cubes from y 0 to 255 for about a minute, but the cubes never held a ticket, were not forced cubes and did not get random ticks

Now a forced column loads and holds the cubes from y 0 to 255, also when the mod forces it again after a restart.

This is meant for migration, giving players time to move to new cube based chunkloaders while their old base still works as before. Its not a permanent solution, loading an entire column just does not work well with CC because columns have no fixed top or bottom

Checklist

  • I have tested this PR in DevEnv
  • I have tested this PR in Fullpack
  • This PR is in compliance with the GTNH AI Policy
  • This PR requires another PR in order to merge

@RecursivePineapple RecursivePineapple left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

good enough, we'll need to add proper chunk loaders (I'd like to make this a mechanic but players may not like it)

@RecursivePineapple
RecursivePineapple merged commit c6685e6 into GTNewHorizons:master Oct 31, 2026
1 check passed
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.

2 participants