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

Fix below-zero-generation-in-existing-chunks #11570

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

notTamion
Copy link
Contributor

fixes #11531
i am not entirely sure what exactly is happening here. my best guess is that the chunk status "full" didn't yet exist in older versions and fully generated chunks were just left on "mobs_spawned".

@notTamion notTamion requested a review from a team as a code owner November 3, 2024 10:29
@kennytv
Copy link
Member

kennytv commented Nov 3, 2024

How exactly did you get to mobs_spawned? The equivalent for full in pre 1.14-ish versions would be fullchunk

@notTamion
Copy link
Contributor Author

notTamion commented Nov 3, 2024

i generated a world in 1.12.2 made sure to wait a bit and also flew around in spectator mode in the ground to make sure everything was generated. i also tried the same in 1.17 and when i checked there the chunks were showing up as full (should note that i used single player to generate this)

@kennytv
Copy link
Member

kennytv commented Nov 6, 2024

Pretty sure if you if you did the same with a 1.14 world it would break again, it's kind of awkward to have the check before datafixing

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

Successfully merging this pull request may close these issues.

Disabling chunk generation below Y=0 does not work
2 participants