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

Stratum ore: Allow use with no noise for simple strata #6480

Closed
wants to merge 1 commit into from
Closed

Stratum ore: Allow use with no noise for simple strata #6480

wants to merge 1 commit into from

Conversation

paramat
Copy link
Contributor

@paramat paramat commented Sep 30, 2017

If either of the 2 noise parameters are omitted the ore will occur from y_min
to y_max in a simple horizontal stratum. As this does not compute noise
performance improves, and is ideal for placing many layers.

Clean up some nearby ore documentation.
/////////////

screenshot_20170930_151228

screenshot_20170930_151344

^ Ore density defined by 'clust_scarcity' is still usable without noise

Tested.

Allows using this ore type with no 2D noises for higher performance, the result is simple horizontal strata with top and base set to ore y limits. These will be useful for various, sometimes unusual, uses.
Stratum ore normally uses 2 2D noises, if either noise parameter is not present in the ore registration no noises are used and the ore occurs between y_min and y_max.

I was inspired by seeing a screenshot of Mineclone2 which had multiple horizontal strata, why insist on 2D noises and compute them when they are not needed?

@paramat paramat added @ Mapgen Feature ✨ PRs that add or enhance a feature WIP labels Sep 30, 2017
If either of the 2 noise parameters are omitted the ore will occur from y_min
to y_max in a simple horizontal stratum. As this does not compute noise
performance improves, and is ideal for placing many layers.

Clean up some nearby ore documentation.
@paramat paramat removed the WIP label Sep 30, 2017
@paramat
Copy link
Contributor Author

paramat commented Sep 30, 2017

Documentation added and tested, now ready.

@paramat
Copy link
Contributor Author

paramat commented Oct 1, 2017

@paramat
Copy link
Contributor Author

paramat commented Oct 1, 2017

e2afcf8

@paramat paramat closed this Oct 1, 2017
@paramat paramat deleted the levelstratum branch October 27, 2017 06:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature ✨ PRs that add or enhance a feature @ Mapgen One approval ✅ ◻️
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant