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

Images corrupted #56

Closed
phunni opened this issue Mar 8, 2018 · 20 comments
Closed

Images corrupted #56

phunni opened this issue Mar 8, 2018 · 20 comments

Comments

@phunni
Copy link

phunni commented Mar 8, 2018

Newly created world is impossible to view and most cells are either called "Unknown RGB" or are listed as something extremely unlikely in that place (as I say, brand new world).

I suspect it's due to an incompatibility with the latest beta version.

download

@phunni
Copy link
Author

phunni commented Mar 9, 2018

I noticed in the final point of the Changes list here that they no longer use Block IDs. Could this be the cause of this bug? would it also mean that mcpe_)viz won't work anymore? I'm really hoping not since I use mcpe_viz extensively with my worlds...

@Plethora777
Copy link
Owner

I'm on the case. It's a fun & maddening investigation.... Mojang did some deep weirdness to save storage space for worlds... Back to it. I'll update when I've got it solved.

@phunni
Copy link
Author

phunni commented Mar 19, 2018

Cool - thanks. As I said, I'm really hoping the changes they've made haven't broken mcpe_viz. I'm so used to using it's hard to imagine Minecraft without it!

@vvanhee
Copy link

vvanhee commented Apr 5, 2018

Just upgraded to 1.2.13 and new saved maps won't work MCPE_viz. When I don't get a crash on start of processing records, I get weird images like posted above. Progress window shows lots of "WARNING: Did not find block variant for block(Mob Head) with blockdata=0 (0x0) MSG3". And lots of other did not find block variant messages for other blocks. My old non-upgraded saved maps still work. I love this program -- anything I can do to help, let me know!

@vvanhee
Copy link

vvanhee commented Apr 5, 2018

You may have already seen this, but if not, it is a description of changes in how blocks are represented in Bedrock now: https://gist.github.com/Tomcc/a96af509e275b1af483b25c543cfbf37

@ynleborg
Copy link

ynleborg commented Apr 5, 2018

Mine is broken too afer updating Realm to 1.2.13 :(

@Thunder33345
Copy link

they are changing it non stop i suggest halting it for some time after it became more stable

@Plethora777
Copy link
Owner

@vvanhee thanks very much for that link. I'd already updated the code to deal with most of the new chunk record stuff. The info in that link should help with the rest (packed bit madness!)

@Plethora777
Copy link
Owner

@Thunder33345 sort of agree, the stuff they are doing.... seems strange. Like writing code for an early computer with 4k of storage strange... It's very flexible but also sort of insane. I don't see the clear benefit of this packed bit business. I guess it saves space, but the cost in, well, sanity is high. And the space savings are lost as soon as you have a sub-chunk with a high number of diff blocks.

@Plethora777
Copy link
Owner

Just pushed an update - give it a try. Please note that --spawnable no longer works because the new paletted chunk format does not store block light info. (This is too bad)

@phunni
Copy link
Author

phunni commented Apr 9, 2018 via email

@vvanhee
Copy link

vvanhee commented Apr 9, 2018 via email

@phunni
Copy link
Author

phunni commented Apr 9, 2018

Just ran it and it seems to have worked, although I get a lot of:

WARNING: getBlockByUname failed to find uname=minecraft:hardened_clay

That's the only one I get now. I was getting loads of different ones before I copied the mcpe_viz.xml. Has the config changed much/at all? I haven't copied it over my existing one, so I'm assuming not since it works.

@Plethora777
Copy link
Owner

@phunni Yep there will be loads of those until we have it covered in the XML file. I added "uname=" for blocks. I'll need to add support for aliases as there seems to be some inconsistencies. I just wanted to get an update out that worked first :) Need to think about how to handle the diff unames

@vvanhee
Copy link

vvanhee commented Apr 10, 2018 via email

@vvanhee
Copy link

vvanhee commented Apr 10, 2018

I can reproduce the problem with bedrock on level 15 using a new map, then every 16th layer has odd issues — starting at level 14 and hitting the up arrow should demonstrate the issue. Layers 31, 47, 63, 79, and 95 seem off compared to layers above and below, with odd pixels of air at regular intervals or other issues. Then going way up, I don’t get 100% air as I think I should, with the top solid blocks not getting erased. I should probably open a new ticket. Thanks!

@Plethora777
Copy link
Owner

@vvanhee Just pushed an update for that issue - thanks for pointing it out

@Shkyrockett
Copy link
Contributor

It looks like there is a new Chunk version in the latest beta.

§3Skyblock §2Challenge.mcworld.zip

@vvanhee
Copy link

vvanhee commented Apr 17, 2018 via email

@Plethora777
Copy link
Owner

I think this may be fixed by the new versions lately. I'm going to close this. Please feel free to open a new one if it's still a problem.

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

No branches or pull requests

6 participants