ddevault / Craft.Net

(Unmaintained, see TrueCraft) Minecraft server, client, and etc for .NET
MIT License
228 stars 65 forks source link

perlin noise based terrain generation, trees, basic biomes #188

Closed tuomas56 closed 11 years ago

tuomas56 commented 11 years ago

Perlin noise based terrain generation with trees (birch and oak) and biomes based on height (beach near or at sea level, extreme hills edge a bit above beach and extreme hills elsewhere) up to ten trees per chunk. stone forms 2/3 of the land with dirt the remaining third. rare gravel generation on the top layer rare clay generation in rivers and sand for beaches and underwater 2013-09-21_11 57 09

ddevault commented 11 years ago

This looks really awesome. This is a feature I've wanted to see implemented for a long time. I have three concerns:

Thanks a ton!

ddevault commented 11 years ago

I removed the XNA dependency locally and nothing broke, for the record.

ddevault commented 11 years ago

It would probably be better to limit the perliny parts to between certain elevations, so that hills aren't super tall and rivers don't reach down to bedrock.

ddevault commented 11 years ago

It's not just the trees that are random, I noticed several places where it's actually random. Please make sure it's entirely procedural.

tuomas56 commented 11 years ago

Removing channels was a complete accident I'm not sure how to fix it or how it happened

On 21 Sep 2013, at 22:15, Drew DeVault notifications@github.com wrote:

It's not just the trees that are random, I noticed several places where it's actually random. Please make sure it's entirely procedural.

— Reply to this email directly or view it on GitHub.

tuomas56 commented 11 years ago

Sorry about the spaces between Coordinates3D and the parentheses it's something Xamarin Studio adds by default (I'm working on a mac now) also with the indentation it all looks in line on my screen, did you mean change the tabs to spaces? On 21 Sep 2013, at 22:15, Drew DeVault notifications@github.com wrote:

It's not just the trees that are random, I noticed several places where it's actually random. Please make sure it's entirely procedural.

— Reply to this email directly or view it on GitHub.

ddevault commented 11 years ago

Yes, please use four spaces instead of tabs.

ddevault commented 11 years ago

Tidied things up and merged it in. Thanks a ton!

tuomas56 commented 11 years ago

Thank you very much!

On 21 Sep 2013, at 23:35, Drew DeVault notifications@github.com wrote:

Tidied things up and merged it in. Thanks a ton!

— Reply to this email directly or view it on GitHub.