IrisShaders / Iris

A modern shaders mod for Minecraft compatible with existing OptiFine shader packs
https://irisshaders.dev
GNU Lesser General Public License v3.0
3.36k stars 641 forks source link

Iris handles datapack dimensions in a way that is inconsistent with both Vanilla and OptiFine #1227

Closed TBlazeWarriorT closed 2 years ago

TBlazeWarriorT commented 2 years ago

What happened?

Screenshots

Dimension Settings for the first 3 screenshots: daytime: 6000 fixed dimension time: none sky effects: the_end skylight: true ambient light: 0 overworld curvature: 1024 end curvature: 0

OptiFine with Shaders Off (reference image) image OptiFine with Complementary (reference image) image Iris with Complementary image Iris 1.18 (the datapack does not work in 1.18 so the biome is different, I just wanted to prove that it would look more or less the the same in 1.18 and wasn't fixed) image

Different-ish setup with Dimension Type set to Overworld and Daytime set to Noon (lots of curvature and lighting/color differences - exact same place and settings on both, only left has curvature) image

Relevant log output

No response

Minecraft Version

1.16.5, 1.18.1

Iris Version

iris-mc1.16.5-1.2.0-pre.jar

Sodium Version

sodium-fabric-mc1.16.5-0.2.0+build.4.jar

Operating System

Windows 10

What is your GPU?

RTX 2060

Java Version

Bundled

Additional context

Not exactly a bug but almost definitely not working how it should or was intended to look like in a perfect scenario. Can cause quite a bit of annoyance and might be hiding one or more bugs behind it.

I tried to include a few decent screenshots but there are just too many variables and having to switch between Iris and OptiFine to compare made it too confusing and hard to track. Hopefully the 3 "issues" above (which kinda are the same) are explained in a decent and helpful way still.

coderbot16 commented 2 years ago

https://github.com/IrisShaders/Iris/commit/7529fea14e261b0f896b6a4808814c5f90882ef1 might fix some of these issues.

coderbot16 commented 2 years ago

@TBlazeWarriorT Can you test again on 1.2.1 / 1.2.2?

TBlazeWarriorT commented 2 years ago

@coderbot16 I just went to the same place in 1.16.5 1.2.2 and after a few seconds I realized it looks great

Overall look: good Dimension Detection for World Curvature: detects as End (good) Ambient Light display: broken (0 and 15 looks identical when shaders are enabled) Fixed Time: does nothing (probably good, end doesn't have daytime?) Normal Maps: working, I think it looks normal to me?

So TL;DR only ambient light seems to be still broken but almost nobody uses it Thx a lot for making my adventure map's end dimension look gorgeous image

coderbot16 commented 2 years ago

Great, thanks for testing, I'll close this as it seems like 1.2.1 resolved it then. Seems like @IMS212's fixes for Terralith's nullscape stuff fixed this too.

TBlazeWarriorT commented 2 years ago

image image I checked another custom dimension from the same map and realized this at the last second, is this enough to get you worried again or worth including on this issue? Even at the default Complementary settings the sky and fog color looks a bit too different from what it should. Other shaders seem to just ignore vanilla sky/fog color so it's a bit harder to test but it only looks farther than vanilla on all other shaders I have

coderbot16 commented 2 years ago

Ultimately that's likely just a stylistic decision by the shader - it can decide to ignore fog colors and sky colors or modify them however it wants. Complementary and BSL take the sky color into account, but they modify it to suit their style.