-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Query the hardware and initialize the settings for the best user experience #15630
Comments
The feature unfortunately causes some mapgen bugs: #9357 |
(Side note: I'm not sure how much of a problem this is with the special case of pure Lua mapgen which does not rely on overgeneration.) |
See #13674 |
What is this 'overgeneration' good for? |
Related #5024
It increases the chunk size that the mapgen is working with (default is 5³ mapblocks = (5*16)³ nodes) with an 1 mapblock wide border so that you can randomly place oversized/overlapping structures (such as trees) into it without instantly risking a cut-off. |
Most mapgens use overgeneration, if this is supposed to be helpful for newbies who just loaded Luanti for the first time, inserting glitches into their brand-new Voxelibre map is gonna give a bad impression and defeat the whole point of the automatic setting initialization. |
Does that imply that when mapgen runs, the chunk (5^3 blocks) might already have trees in it at the bottom? This looks suspicious to me: The neighboring chunks need to be locked as well I guess since it is not just one chunk that is written to. I also don't get, why this function should be multi threaded? |
Couple of observations from my setup (with sqlite):
Maybe things are different with different DB backends. |
Problem
It is amazing how much the user experience improves when increasing the value of
num_emerge_threads
.The sad thing is that I didn't know about this setting until recently.
Solutions
Query the hardware and initialize the settings for the best user-experience.
Alternatives
Don't hide
num_emerge_threads
Additional context
No response
The text was updated successfully, but these errors were encountered: