sbrl / Minetest-WorldEditAdditions

Extra tools and commands to extend WorldEdit for Minetest
https://worldeditadditions.mooncarrot.space/
Mozilla Public License 2.0
16 stars 3 forks source link

Toggle command execution conformation #30

Open VorTechnix opened 3 years ago

VorTechnix commented 3 years ago

I'd like to request the addition of a config/settings file to this mod. Further I'd like to request the option to turn off command execution conformation for large operations (requiring //y///n when a large number of blocks will be effected) be added to the settings file.

sbrl commented 3 years ago

Hey there! Unfortunately, that's really only something that can be implemented in WorldEdit itself, and could be potentially rather dangerous. However, you can use the //multi command as a workaround like this:

//multi //set dirt //y

For //subdivide I've already implemented logic to bypass saferegion functionality (the internal name for the //y execution confirmation), so I could potentially implement an //unsafe command like this:

//unsafe set dirt

....but I'm wary of doing that, because the safe region functionality is there to avoid awkward issues, and I've rather avoid encouraging people to accidentally crash their machines :P

Note that for large regions, in the latest master //subdivide is now optimised for extremely large regions (>= 500 million nodes). It took some doing, but I'm rather pleased with the result (though servers with limited amounts of RAM tend to struggle at that scale).

VorTechnix commented 3 years ago

Sorry, didn't know it was a feature of the main mod... Forget I said anything.

sbrl commented 3 years ago

No worries, @VorTechnix! I'll close this issue for now, but I'm happy to reopen if there's anything further to discuss here.

VorTechnix commented 2 years ago

Now that we're implementing WEA Core I think we should look into this again. With core control this would cease to be an upstream issue.

sbrl commented 2 years ago

Absolutely. given we have our own safe_region implementation now, this should not be that much of a problem.

We have yet to implement a storage interface for persisting things though, so that part may have to wait a bit longer.