Skip to content

Conversation

@ChickingWasTaken
Copy link

Uses perlin noise for dirt generation so it is nicer to look at

@ChickingWasTaken ChickingWasTaken changed the title added tile_picker feature for better dirt generation added tile_picker feature for better dirt generation in diggy Apr 11, 2019
local Command = require 'utils.command'
local CreateParticles = require 'features.create_particles'
local Ranks = require 'resources.ranks'
local TilePicker = require 'map_gen.maps.diggy.feature.tile_picker'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you disable the feature, what happens?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It generates terrain like it used to if not enabled then return 'dirt-' .. random(1, 7) end

@mheguy mheguy added the Diggy The Diggy scenario label Apr 16, 2019
@theorangeangle
Copy link
Member

@linaori was wondering if i could get a final yes/no on this PR, so that we can clean out some of the old ones!

@BlueRock2
Copy link
Contributor

BlueRock2 commented Feb 17, 2020

I know this is an old PR but wanted to give some feedback. I like the option to use perlin noise for the floor tiles. Just a little sad that the changes create a new tile_picker.lua file to generate the terrain. Unfortunately, this moves a little further away from the wishlist item (Diggy feedback/todo collection thread #926) to try and use ore generation system (which is rather robust) and maybe expand it to be able to handle the noise for terrain and rooms (which also use noise) as well. Granted, the wishlist item came out shortly after this PR was made.

@BlueRock2
Copy link
Contributor

Or even better, use when possible tools that are shared by other RedMew maps. Or add to shared tools if other maps could benefit. (I still need to learn a lot about what is exactly is available right now)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Diggy The Diggy scenario

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants