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

Stairs next to stairs still exhibit incorrect shadow behavior (more obvious when smooth lighting is turned off) #13

Open
1 of 2 tasks
ninjakitty844 opened this issue Sep 15, 2023 · 10 comments
Labels

Comments

@ninjakitty844
Copy link

ninjakitty844 commented Sep 15, 2023

Describe the bug

2023-09-14_22 25 56

2023-09-14_22 30 44

Reproducibility

Please try to reproduce the issue with as few other mods as possible. Then check one of the following checkboxes according to your results. To check a checkbox, replace the space between the square brackets with an x (like this [x]) or create the issue and check the checkbox by clicking on it.

  • I reproduced this issue with as few other mods as possible installed. (only Red Core, MixinBooter, ConfigAnytime, Valkyrie, and Alfheim)
  • I am unable to reproduce this issue consistently.

To Reproduce

  1. Put stairs in a pattern similar to the ones shown in my screenshots

Expected behavior

I think most stair lighting issues would be solved if stairs were prevented from blocking light under all circumstances, I believe that is how modern versions of Minecraft handle stairs.

@Desoroxxx
Copy link
Member

would be solved if stairs were prevented from blocking light under all circumstances
That would literally prevent them from being used as a roof

would be solved if stairs were prevented from blocking light under all circumstances
No

Concerning how to fix this there probably is a way but it isn't my priority to work on this issue, everything I can think of as drawbacks.
Also, this is with smooth lighting off, smooth lighting on probably fixes the issue anyway

@ninjakitty844
Copy link
Author

Might be that modern stairs only block light on their bottom side and none of the other sides.

idk exactly how it works, but I'd look to modern MC to see how they fixed the issue.

@ninjakitty844
Copy link
Author

And mooth lighting only hides the problem a little bitz you can still tell the issue is there with it.

@Desoroxxx
Copy link
Member

Might be that modern stairs only block light on their bottom side and none of the other sides.

That would not fix the issues you showed

@XvaleX10
Copy link

XvaleX10 commented Mar 4, 2024

I have the same issue
without the mod
2024-03-04_15 06 48
with the mod
2024-03-04_15 11 34

@Desoroxxx
Copy link
Member

I have the same issue without the mod 2024-03-04_15 06 48 with the mod 2024-03-04_15 11 34

Oh yeah that's really not right, I will try to take a look.

@MrRedstoneA1
Copy link

MrRedstoneA1 commented May 5, 2024

Also experiencing issues with stairs.
image
Oh, it also affects slabs but I don't have a screenshot of that atm.

@xskutsu
Copy link

xskutsu commented Dec 24, 2024

Would like to add that it doesn't need to be stairs next to stairs. I'm able to replace the issue with one stair alone. This looks very distracting in any build that uses upside down stairs.
image

Also the problem still persists with smooth lighting enabled, it is just harder to notice.
image

@Desoroxxx
Copy link
Member

Desoroxxx commented Dec 24, 2024

I will take a look at this in the next release

@touhouzun
Copy link

touhouzun commented Dec 27, 2024

Would like to add that it doesn't need to be stairs next to stairs. I'm able to replace the issue with one stair alone. This looks very distracting in any build that uses upside down stairs. image

Also the problem still persists with smooth lighting enabled, it is just harder to notice. image

yeah dude,in your first picture,the shadow is not fake zero(light level,for client),but real zero(for server),which turns the grass block underneath into dirt.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants