mojira.dev
MC-75031

"cullface": "down" does not always hide faces.

json command:
"cullface": "down"
does not hide the face it should affect, when the model itself is not a solid block.

So if you place a Grass block on top of another Grass block it works. But if u place a nonsolid block like Brick Stairs on top of a Grass block, the faces flagged as cullface:down are not hidden on the Stairs.

Attachments

Comments 10

The syntax is:

cullface:"down"

not:

"cullface":"down"

Repeaters and comparators are especially finicky with cullface, see MC-73348

The syntax is "cullface":"down" as far as I know. That is what is used in the default models. And it works... Well, most of the time 😉

I haven't tried with repeaters, or most other blocks. But yes, they could be related somehow. Though MC-73348 is the opposite problem.

Yeah. Skipping quotes around cullface causes a parsing error.

Is this still an issue in the current Minecraft Snapshot 15w45a or later? If so, please update the affected versions in order to best aid Mojang ensuring bugs are still valid in the latest releases/pre-releases.

Well it partly works. Maybe I should have been more specific. The stairs was only an example. It's not working for many (all?) non-solid blocks.

So with snapshot 15w45a, stairs seams to works. But e.g. Slabs (lower half) does not work still.

I would have thought it was a general routine for all non-solid blocks that was in need of a fix. And thus fixing it all. But I guess I was wrong there.

Would it help if I compiled a list of all affected blocks? It would be a lot of work to do so. And I didn't do it, as I am expecting all non-solid blocks to be affected.

Keep up the good work 🙂

@unknown, no need to compile a list, but can you please attach some screenshots and mark the affected faces.
BTW: You can update the "Affected Versions" since you are the creator of this ticket.

This is sadly how the system internally works. The actual culling of faces is not decided by the model but by custom code. You can only flag the faces that would actually cull if the code decides this should be done.

No way to change this to something sufficiently flexible without a major internal overhaul and exposing a lot of logic into models.

Okay. Thanks for at least taking a look at this.

cheers

Mogens Sørensen

Erik Broes

Unconfirmed

Minecraft 1.8.1, Minecraft 1.8.8, Minecraft 15w45a

Retrieved