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

Lucent lighting flickers slightly #236

Open
KittyNath opened this issue Mar 17, 2024 · 2 comments
Open

Lucent lighting flickers slightly #236

KittyNath opened this issue Mar 17, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@KittyNath
Copy link

Bug Description

Lucent provides dynamic lights (light emitted from holding a torch, etc). With Embeddium installed, the dynamic light source snaps to center of the block instead of following the player smoothly. This behaviour doesn't change with adding Embeddium++ and Oculus. Lucent settings: smoothBlending enabled, threadedRendering both enabled and disabled tested

Reproduction Steps

  1. Install both Lucent and Embeddium on Forge 1.20.1
  2. hold a torch in a dark cave
  3. walk around and observe the jank

Log File

latest.log
debug.log

@KittyNath KittyNath changed the title Incompatibility with Lucent Incompatibility with Lucent [Forge 1.20.1] Mar 17, 2024
@embeddedt embeddedt changed the title Incompatibility with Lucent [Forge 1.20.1] Lucent lighting flickers slightly Mar 17, 2024
@embeddedt
Copy link
Owner

embeddedt commented Mar 17, 2024

This is likely caused by Lucent using 8 bits to store the rendered light value. Vanilla only uses the top 4 bits, and many of the functions in vanilla that interact with the lightmap also assume 4-bit values.

This cannot be fixed before we stop remaining compatible with Sodium 0.5.8 addons, as it requires making a major change to Embeddium's light code that will break Indium at the very least, and likely other add-ons.

In my testing with a standalone instance, the light blending is still smooth enough to be tolerable even with this limitation.

@embeddedt embeddedt added the enhancement New feature or request label Mar 17, 2024
@embeddedt
Copy link
Owner

I spent some more time investigating this today, and I'm not sure it's worth fixing, especially since Forge's own lighting pipeline (when enabled) also makes the assumption that only 4 bits matter, and thus has the same flickering problem. If we do opt to fix this, it will likely be in 1.21 or later.

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

No branches or pull requests

2 participants