Skip to content

Conversation

douira
Copy link
Collaborator

@douira douira commented Sep 12, 2025

Remove redundant code for implementation of the sort task defer mode.

There was previously an implementation for controlling sort task deferring as part of the sorting code. A different implementation of essentially the same concept but for rebuild tasks was added later. Now they both use the newer approach, which simplifies the task submit code.

Improve immediate presentation of meshes generated in blocking mode.

Previously even when meshes were generated in the blocking defer mode "Immediate" (zero frame) they would not become visible if they were part of a previously invisible section and thus not in a render list already. This change makes it so that sections that receive results from blocking rebuilds are added to the render list.

When the "Immediate" chunk update defer mode is selected this fixes the issue of holes in the world when breaking a block that reveals a previously invisible section without needing to perform a second graph search pass. This is especially an issue on slower hardware and lower fps limits.

There was previously an implementation for controlling sort task deferring as part of the sorting code. A different implementation of essentially the same concept but for rebuild tasks was added later. Now they both use the newer approach, which simplifies the task submit code.
@douira
Copy link
Collaborator Author

douira commented Sep 12, 2025

This is currently broken

@douira douira marked this pull request as draft September 12, 2025 18:11
Previously even when meshes were generated in the blocking defer mode "Immediate" (zero frame) they would not become visible if they were part of a previously invisible section and thus not in a render list already. This change makes it so that sections that receive results from blocking rebuilds are added to the render list.
@douira douira force-pushed the improve-immediate-defer-handling branch from 75ea1e8 to 8b10c2d Compare September 12, 2025 19:26
@douira
Copy link
Collaborator Author

douira commented Sep 12, 2025

It appears I've fixed it. Briefly tested by Adre and me. It needed more code than I wanted but ok, it seems to work.

@douira douira marked this pull request as ready for review September 12, 2025 19:36
@douira douira changed the title Improved Non-Deferred Chunk Presentation Improved Immediate Chunk Presentation Sep 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant