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

Incorrect shadows #22

Closed
StealWonders opened this issue Jun 14, 2020 · 6 comments
Closed

Incorrect shadows #22

StealWonders opened this issue Jun 14, 2020 · 6 comments
Labels
T-bug Type: Bug

Comments

@StealWonders
Copy link

Expected Behavior

Expected should be that the shadows either be darker than normal or just no shadows.
See this screenshot:

javaw_6H9iexObun

Actual Behavior

Sodium-fabric causes the shadows of the trees and around the house to be lighter than the actual normal lighting... This looks very unrealistic :(
Screenshot:

javaw_N7Ve4BOPqH

Reproduction Steps

Build a tree and look at its shadows, generate a vanilla world, and look at those shadows.
This issue is with vanilla generated worlds too. Depending on biome it looks worse or less.

System Information

  • Java Version: 1.8.0_51 64bit
  • CPU: Intel(R) Core(TM) i7-4790K
  • GPU: GeForce GTX 1080

Enabled in the mod:

  • Chunk Renderer: OpenGL 4.6
@StealWonders StealWonders added the T-bug Type: Bug label Jun 14, 2020
@jellysquid3
Copy link
Member

Sodium re-implements the entire smooth lighting engine in order to improve chunk rendering performance and to fix a few graphical issues. As a result, the rendering between the two will not always be identical.

However, it is rather difficult for me to tell the difference between these screenshots as they've been compressed heavily and the camera moves between each shot. If you're able to, cleaner images would help a lot in diagnosing this problem.

@jellysquid3 jellysquid3 added the S-info-needed Status: Info needed label Jun 21, 2020
@lenrik1589
Copy link

It might not be obvious as the marking is not clear but if you look near big outline just on the inner side some blocks look de-saturated/brightened
in this area:
sodium#22_area_selected
Alot more contrast:
sodium#22_contrast_a_bit_more_sane
Edge here:
sodium#22_contrast_a_bit_more_sane_edge
this should make it a bit clearer

@StealWonders
Copy link
Author

Sorry for the images but the issue is hardly noticeable.

No sodium:
2020-06-21_19 21 50

With sodium:
You may be able to tell that the wall of the house is outlined by a two block wide lighter grass strip.
2020-06-21_19 21 03

Image highlighting the border of where the shadows are to make it more clear.
ApplicationFrameHost_H7wkqAit9E

Light level near the house (using Vanilla):
javaw_XYtrrbdXjh

Light level further away from the house (using Vanilla):
javaw_mYKfgFa0hL

The light levels are the same when using sodium of course but the coloring of the grass (shadows) is brighter than normal grass which makes no sense...

@jellysquid3
Copy link
Member

Are you using the latest revision of Sodium? There were some issues in an earlier version with the vertex encoding stage that caused weirdness like this.

@StealWonders
Copy link
Author

Are you using the latest revision of Sodium? There were some issues in an earlier version with the vertex encoding stage that caused weirdness like this.

Let me recompile. Here are some further screenshots of the issue for now:

javaw_I1QwmFSH4n

javaw_qXMnOiGbu8

@StealWonders
Copy link
Author

Issue is non-existent on the latest commit as of now. I'ma close this!

@amnotbananaama amnotbananaama removed the S-info-needed Status: Info needed label Dec 12, 2020
MATRIX-feather pushed a commit to MATRIX-feather/sodium-fabric that referenced this issue Jul 18, 2021
…-sections-for-rebuilds

Do not queue RenderSections for rebuilds during shadow rendering
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-bug Type: Bug
Development

No branches or pull requests

4 participants