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

Biome-blending cannot be disabled for dark leaves #2273

Closed
3 tasks done
Lolothepro opened this issue Sep 29, 2024 · 3 comments
Closed
3 tasks done

Biome-blending cannot be disabled for dark leaves #2273

Lolothepro opened this issue Sep 29, 2024 · 3 comments

Comments

@Lolothepro
Copy link

Lolothepro commented Sep 29, 2024

NeoForge Version

21.1.64

Twilight Forest Version

4.6.2790

Client Log

https://mclo.gs/Az2icCw

Crash Report (if applicable)

No response

Steps to Reproduce

Go to the border between the “dark_forest” biome and the “dark_forest_center” biome
Press Esc
Go to video settings and set “Biome Blending” to “Off”.

What You Expected

No biome-blending

What Happened Instead

The color of the leaves is still blended

Additional Details

2024-09-29_12 03 32

Please Read and Confirm The Following

  • I have confirmed this bug can be replicated without the use of Optifine.
  • I have confirmed the details provided in this report are concise as possible and does not contained vague information (ie. Versions are properly recorded, answers to questions are clear).
  • I have confirmed this issue is unique and has not been reported already.
@Lolothepro Lolothepro added the bug label Sep 29, 2024
@Andromander
Copy link
Member

This appears to be how Dark Leaves - and to this extent, Giant Leaves - are tinted.

Long story short: the blend radius setting is working.

Short story long: It works, but only if we just got the average foliage colour and called it a day. Instead, an average is picked from a small radius around the block itself. That basically means blending is always present, but if "disabled", the blending is only slightly less so than if it were enabled. It appears to be a 1-block radius from the center, so an average of all the foliage colour in a 3x3 area.
As for why, I can't really explain it, tbh. Someone else that may have ported the mod to 1.12 might know.

No Biome Blending:
2024-10-12_21 52 47

Normal Biome Blending:
2024-10-12_21 52 56

@Andromander Andromander removed the bug label Oct 12, 2024
@GizmoTheMoonPig
Copy link
Member

I went all the way back to 1.12 to check this out and apparently our code is just a copy-paste job of how BiomeColorHelper.getColorAtPos used to work. I dont see any need for us to keep doing what we're doing when we can just use vanilla's methods for blending. I'll be fixing this soon

@GizmoTheMoonPig
Copy link
Member

also worth noting this affects every single one of our leaves, not just dark

GizmoTheMoonPig added a commit that referenced this issue Dec 26, 2024
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

3 participants