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

[WIP] Add small hills to mapgen #36605

Closed
wants to merge 2 commits into from
Closed

[WIP] Add small hills to mapgen #36605

wants to merge 2 commits into from

Conversation

arijust
Copy link
Contributor

@arijust arijust commented Dec 31, 2019

Summary

SUMMARY: Content "Add small hills to mapgen"

Purpose of change

Add some hills to wilderness, so map doesn't feel flat outside of cities.

Describe the solution

Add small hills with some vegetation.

Describe alternatives you've considered

Add one more Z-level to it.

Additional context

Please mention, if I am missing something like some parts for overmap settings.

@Amoebka
Copy link
Contributor

Amoebka commented Jan 1, 2020

The hill has perfectly square cross-sections on every z-level and no variants. It looks more like a ziggurat than a hill.
Hills should probably also be their own overmap terrain type on the same level as forests, plains and swamps, filled using a noise algorithm instead of randomly sprinkling them with no pattern.

@ghost
Copy link

ghost commented Jan 1, 2020

Please don't try to turn Cata into Dwarf Fortress tier abomination of z-levels bonanza.

IMO the 'flat' terrain should be interpreted as ground level, no matter the elevation - which probably should be property for any such ground tier tile and any related mechanics like stamina/fuel drain or visibility modifies implemented from that, reserving up/down transitions to represent sheer drops, stairs etc.

@arijust
Copy link
Contributor Author

arijust commented Jan 1, 2020

Closing.
Other contributors can make these "in correct way" themselves.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants