Glowstone-Legacy icon indicating copy to clipboard operation
Glowstone-Legacy copied to clipboard

Add overworld world generator

Open jimmikaelkael opened this issue 9 years ago • 14 comments

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:

  • Ticket: #76

jimmikaelkael avatar Jan 11 '15 17:01 jimmikaelkael

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:

  • The entire ocean floor seems to be gravel (with stone only one layer below). I think this should be dirt instead, and for the first several layers instead of just one, to match vanilla more closely. Thoughts?
  • DungeonPopulator's method for picking random tools is commented as if it expects index to be in the [0,3] range, but is called with a 4 index, leading to items with odd rarities:
    • Diamond rarity: STICK
    • Gold rarity: STRING
    • Iron rarity: FLINT_AND_STEEL
    • Stone rarity: DIAMOND_SWORD
    • Wood rarity: STONE_SWORD

SpaceManiac avatar Jan 13 '15 04:01 SpaceManiac

@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:

  • Mesa doesn't have colored stripes (almost done)
  • Bryce doesn't have spikes (almost done)
  • Mesa forests doesn't have grass and coarse dirt (almost done)
  • Switch to Simplex noise for flowers
  • Needs several layers of dirt below ocean gravel as you mentionned
  • In roofed forest there's not enough tall grass on ground (due to tree density)
  • In jungles there's not enough ground bushes
  • Caves and Ravine generator isn't done (not planned for this PR)
  • Block lighting isn't working and once operational it should overall look better (Not in the scope of this PR)
  • Biome temperature is not handled so water placed by the generator is never turned into ice (Not in the scope of this PR)

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 avatar Jan 13 '15 10:01 jimmikaelkael

@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.

jimmikaelkael avatar Jan 13 '15 20:01 jimmikaelkael

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).

SpaceManiac avatar Jan 13 '15 21:01 SpaceManiac

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

  • Caves and Ravine generator isn't done
  • Block lighting isn't working and once operational it should overall look better

jimmikaelkael avatar Jan 18 '15 08:01 jimmikaelkael

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

gdude2002 avatar Jan 20 '15 09:01 gdude2002

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

jimmikaelkael avatar Jan 20 '15 09:01 jimmikaelkael

<3

gdude2002 avatar Jan 20 '15 09:01 gdude2002

I think it's ready.

jimmikaelkael avatar Jan 30 '15 17:01 jimmikaelkael

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

Tonodus avatar Jan 30 '15 18:01 Tonodus

Merge would be awesome.

bennasar99 avatar Feb 08 '15 18:02 bennasar99

Is there anything holding this back? @SpaceManiac?

gdude2002 avatar Feb 27 '15 20:02 gdude2002

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

jimmikaelkael avatar Feb 27 '15 20:02 jimmikaelkael

@gdude2002 Pending review.

turt2live avatar Feb 28 '15 02:02 turt2live