Skip to content

Blocks above y=254 do not render #2003

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

Closed
jack-webb opened this issue Dec 7, 2021 · 7 comments
Closed

Blocks above y=254 do not render #2003

jack-webb opened this issue Dec 7, 2021 · 7 comments

Comments

@jack-webb
Copy link

jack-webb commented Dec 7, 2021

I think related to #1954, blocks at or above 255 aren't rendering on my 1.18 world. I'm using master a5b36f5. This also doesn't render below y=0

Render In-game
UUIsA 4f9NA
@jack-webb jack-webb changed the title Blocks above y=255 do not render Blocks above y=254 do not render Dec 8, 2021
@greatmastermario
Copy link
Contributor

greatmastermario commented Dec 13, 2021

Looks like there are some hardcoded references to max height (formerly 255) and/or max y for sections. Section data can now go as low as -4 and as high as 19 (inclusive), so any references to these boundaries will need to be looked for. I am taking a look, and if I find all the references I will open a PR. If anyone can beat me to the punch please feel free

@greatmastermario
Copy link
Contributor

#2012

I got the fix working. Also fixes Issue 1954

@arcctg
Copy link

arcctg commented Dec 31, 2021

@greatmastermario sorry to bother you. How I can use your bugfix/2003_1.18_Increased_Height with fixed render above 254 height to render my world?

@greatmastermario
Copy link
Contributor

greatmastermario commented Dec 31, 2021

@Daanay After you check out my branch, you need to recompile the C libraries. Follow the instructions in the documentation to do this. I did it with Python 3.9 with option 2 for compiling, but you should be able to use whatever works for you and the platform you are compiling on. Once that's done, run as normal.

Also, my branch is on a fork, so you may need to clone my fork of the repo to access the branch. I've never tried to checkout from a fork before.

@arcctg
Copy link

arcctg commented Dec 31, 2021

Thank you so much!

@jobe1986
Copy link

This bug does make me wonder about custom dimension types, which if I read correctly can have Y coordinates ranging from -2032 to 2031 (looking at min_y and height from custom dimension types on https://minecraft.fandom.com/wiki/Custom_dimension?so=search#Dimension_type ) Though as yet I havent tested how accurate that info is on the available options.

@greatmastermario
Copy link
Contributor

Since this issue was fixed with my PR and merged to the main branch, can we close this issue?

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

No branches or pull requests

5 participants