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

Remove indev mapgen #1407

Closed
wants to merge 2 commits into from
Closed

Remove indev mapgen #1407

wants to merge 2 commits into from

Conversation

proller
Copy link
Contributor

@proller proller commented Jun 22, 2014

Anything can be done via lua mods

@sfan5
Copy link
Member

sfan5 commented Jun 23, 2014

There is no replacement mapgen for indev in Lua.

@SmallJoker
Copy link
Member

^ true. Also, Lua is a way too slow for "complicated" operations.

@paramat
Copy link
Contributor

paramat commented Jun 23, 2014

Yes, please remove indev mapgen. A replacement would be easy, choose a floatlands mod, choose a cavern mod, use mapgen settings in minetest.conf to create higher mountains. Floatland and cavern lua mods are simple and therefore fast in lua (1 second per chunk non luaJIT on an old laptop).
Currently in indev, the floatlands are flat, limited and boring (i designed them a year ago), the big caves are sliced by the chunk edges and have flat sides, very poor quality.

@paramat
Copy link
Contributor

paramat commented Jun 26, 2014

Fast mod to replace indev's big caves https://forum.minetest.net/viewtopic.php?f=9&t=9601 roughly half a second per chunk.

@sapier sapier added this to the 0.4.11 milestone Jun 28, 2014
@CasimirKaPazi
Copy link
Contributor

I'm also ok with removing it. But what happens when someone loads a world that previously used indev mapgen? I don't see anywhere where it switches to v6.

Conflicts:
	builtin/mainmenu/dlg_create_world.lua
	minetest.conf.example
@proller
Copy link
Contributor Author

proller commented Jul 12, 2014

2d90aca

@proller proller closed this Jul 12, 2014
@sfan5 sfan5 modified the milestone: 0.4.12 Dec 26, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants