Skip to content

Light Generation#32

Open
teknostom wants to merge 20 commits intoSteel-Foundation:devfrom
teknostom:dev
Open

Light Generation#32
teknostom wants to merge 20 commits intoSteel-Foundation:devfrom
teknostom:dev

Conversation

@teknostom
Copy link
Copy Markdown
Contributor

Adds block states that will be needed later. For now, we need it for the opacity.

Adds sky light initialization to the generation steps. Sets block light to 0, as it does not need to be 15 for fullbright.

No propagation yet, but we also don't save the chunks, so no need to do propagation, becasue the client handles that fine on it's own. Once we store chunks, we can impl the light spreading.

@teknostom
Copy link
Copy Markdown
Contributor Author

Reduced the added json from 150k to 64k

@teknostom teknostom marked this pull request as draft December 6, 2025 11:20
@teknostom teknostom marked this pull request as ready for review December 6, 2025 12:05
@teknostom teknostom marked this pull request as draft December 6, 2025 16:54
@teknostom
Copy link
Copy Markdown
Contributor Author

This now works fully seemingly, but the generation tasks for light takes anywhere from 4-70ms. So i will look into how starlight and other light optimization mods try to improve the performance.

@tn-lorenz
Copy link
Copy Markdown
Contributor

Wouldn't it be possible to SIMD iff the light doesn't cross a chunk border? I'm asking because the offsets are vectors

@teknostom teknostom changed the title add SkyLight init to Generation Light Generation Dec 19, 2025
@teknostom
Copy link
Copy Markdown
Contributor Author

Found some small optimizations. Light is now 100-200x faster, and is now in the microseconds for most chunks. Might be more to squeeze out.

@4lve
Copy link
Copy Markdown
Collaborator

4lve commented Dec 20, 2025

Nice, I heard they rewrote the whole light engine in the new snapshot, could you take a look at that?

@teknostom teknostom changed the base branch from dev to master December 20, 2025 20:29
@teknostom teknostom changed the base branch from master to dev December 20, 2025 20:31
@teknostom teknostom marked this pull request as ready for review December 21, 2025 21:54
@4lve
Copy link
Copy Markdown
Collaborator

4lve commented Feb 4, 2026

As we discussed on Discord this will have to wait until other parts of chunkgen are done first

@github-actions
Copy link
Copy Markdown

This pull request has conflicts with the base branch "master". Please resolve those so we can test out your changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants