GlowstoneMC / Glowstone-Legacy

An open-source server for the Bukkit Minecraft modding interface
Other
363 stars 122 forks source link

Add overworld world generator #607

Open jimmikaelkael opened 9 years ago

jimmikaelkael commented 9 years ago

This merges PR #572 and #597 and add one commit on top of it: https://github.com/GlowstoneMC/Glowstone/commit/888f7846be0727f1b74da6172811428fa0cc1742

You can see the results here: http://imgur.com/a/huhmT


Related Links:

SpaceManiac commented 9 years ago

I really love what you've done! The terrain generation looks just fantastic. I've reviewed the implementation a little bit and so far have a few questions:

jimmikaelkael commented 9 years ago

@SpaceManiac OK I'll try to add several layers of dirt below ocean gravel. For dungeons, I think what you're talking about is in the actual implementation (which is not interfered with in this PR) which should be replaced once PR #564 pulled and so have the correct chest loots after that.

It still WIP anyway as I need #572 and #597 to be pulled first: it will let me some time to fix a few problems:

Another thing that could be cool to refine world gen would be to have AMIDST compatible with Glowstone so that we can check the world map in real time.

jimmikaelkael commented 9 years ago

@SpaceManiac I checked it once again on a fresh vanilla 1.8 generated map in solo and it's stone directly under gravel in oceans. Possibly you can expect to find some dirt patches here and there because it's populated in stone as an ore.

SpaceManiac commented 9 years ago

Ah, it would seem you are correct on both counts. I was operating on outdated knowledge wrt the ocean floor - you can keep it as-is - and indeed your PR does not touch the DungeonPopulator (I had been reviewing the entire generator package).

jimmikaelkael commented 9 years ago

I think I have handled most of the problems mentionned above, leaving 2 of them for another PR:

gdude2002 commented 9 years ago

This looks pretty fantastic, IMO. Is it done? Can't wait to have this in master ._.

jimmikaelkael commented 9 years ago

I still need to adjust a few things. I'll notify when it's ready which should happen in the coming days ;)

gdude2002 commented 9 years ago

<3

jimmikaelkael commented 9 years ago

I think it's ready.

Tonodus commented 9 years ago

I've just reset my world to test your PR.... you've definitely done awesome work! :+1:

bennasar99 commented 9 years ago

Merge would be awesome.

gdude2002 commented 9 years ago

Is there anything holding this back? @SpaceManiac?

jimmikaelkael commented 9 years ago

The two other PRs I guess, #572 and #597.

turt2live commented 9 years ago

@gdude2002 Pending review.