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

[Bug] Fast Leaves are broken #132

Open
MetalKnight56 opened this issue Oct 3, 2024 · 10 comments
Open

[Bug] Fast Leaves are broken #132

MetalKnight56 opened this issue Oct 3, 2024 · 10 comments
Assignees
Labels
bug Something isn't working

Comments

@MetalKnight56
Copy link

Basically when enabling fast leaves, instead of making their transparent parts opaque/darken, they continue to be transparent, but without rendering the other side.

With Fast graphics:
image

Without
image

Expected behaviour:
image

@JT122406 JT122406 self-assigned this Oct 6, 2024
@JT122406 JT122406 added the bug Something isn't working label Oct 6, 2024
@JT122406
Copy link
Member

JT122406 commented Oct 6, 2024

I will have to dive into this since we just set the render type with Mc normally it handles the rest so idk if it’s a midlander issue or something, what platform and Mc version are you using/have you tried any other platform or version

@MetalKnight56
Copy link
Author

I will have to dive into this since we just set the render type with Mc normally it handles the rest so idk if it’s a midlander issue or something, what platform and Mc version are you using/have you tried any other platform or version

I've tried using the default forge launcher, on 47.2.20, 1.20.1. Switched launchers and also disabled all mods aside OhTheBiomes...

I've actually found out a bit about why this issue happens, apparently bwg doesn't define the rendering method in the block model for it's leaves. Take a look at how acacia leaves are structured:

{
  "parent": "minecraft:block/leaves",
  "textures": {
    "all": "minecraft:block/acacia_leaves"
  }
}

After i added that in a resourcepack it kinda works, but the thing is, the inbetweens (where it's currently transparent), become all black and not exactly a dark greenish as other leaves, so yeah, that's all i could gather out of it

@JT122406
Copy link
Member

JT122406 commented Oct 6, 2024

We do the samething as normal mc
{
"parent": "minecraft:block/leaves",
"render_type": "minecraft:cutout_mipped",
"textures": {
"all": "biomeswevegone:block/fir/leaves"
}
}
We also add in the render type in json as that's the way forge wants you to do rendertypes

@MetalKnight56
Copy link
Author

We do the samething as normal mc

{

"parent": "minecraft:block/leaves",

"render_type": "minecraft:cutout_mipped",

"textures": {

"all": "biomeswevegone:block/fir/leaves"

}

}

We also add in the render type in json as that's the way forge wants you to do rendertypes

That's odd, hmm are you able to replicate this issue in your side ? Maybe it's something on my end

@JT122406
Copy link
Member

JT122406 commented Oct 7, 2024

So I can recreate it on NeoForge and Forge but on fabric it works fine which leaves me to believe it's an issue with how Forge like platforms handle the rendertype changing during graphic settings changes

@JT122406
Copy link
Member

JT122406 commented Oct 7, 2024

I'm gonna close this as there's nothing we can do on our side unless we use the Deprecated by forge way of using the vanilla map. This should be reported to forge as a issue with the rendertype not changing with the graphics properties

@JT122406 JT122406 closed this as not planned Won't fix, can't repro, duplicate, stale Oct 7, 2024
@MetalKnight56
Copy link
Author

I'm gonna close this as there's nothing we can do on our side unless we use the Deprecated by forge way of using the vanilla map. This should be reported to forge as a issue with the rendertype not changing with the graphics properties

anything i can do? Should i update forge or something like that?

@JT122406
Copy link
Member

JT122406 commented Oct 7, 2024

No It just needs to be fixed by the forge team

@Onigeshi
Copy link

Onigeshi commented Feb 6, 2025

Hi, first of all sorry for "reviving" this, but I've encountered this recently as well and started looking to see if I can fix it with a resourcepack.. and it looks like removing the "render_type": "minecraft:cutout_mipped", from the models fixes it, at least with NeoForge on 1.21.1 (ver. 21.1.115). I haven't tested with fabric or other versions.

I'm not a modder but reading the documentation it looks like this field is optional and so far flying around the world I don't see any rendering weirdness of the leaves after removing this field.

https://docs.neoforged.net/docs/resources/client/models/#render-types

fast (tested using aspen leaf):
Image

fancy (tested using aspen leaf):
Image

fast settings flying over an entire zelkova forest after removing the render_type field:
Image

Edit: after more testing, the only leaves having issues by removing the render_type are the Yucca Leaves

Image

@JT122406
Copy link
Member

JT122406 commented Feb 7, 2025

NeoForge has changed somethings so I will look into it on there side, Forge I am working with on Fixing that can be found here and will be fixed once this pr is merged
1.21.1 MinecraftForge/MinecraftForge#10396
1.20.1 MinecraftForge/MinecraftForge#10393

@JT122406 JT122406 reopened this Feb 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants