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

Vertical biome blend: Tune PRNG seed for finer detail #7329

Merged
merged 1 commit into from May 14, 2018
Merged

Vertical biome blend: Tune PRNG seed for finer detail #7329

merged 1 commit into from May 14, 2018

Conversation

paramat
Copy link
Contributor

@paramat paramat commented May 13, 2018

screenshot_20180513_211553_m0p9

screenshot_20180513_210455_m0p9

^ PR

screenshot_20180513_183952_or

^ Master

@paramat paramat added @ Mapgen Maintenance Tasks to keep the codebase and related parts in order, including architectural improvements labels May 13, 2018
@SmallJoker
Copy link
Member

How about a value in between of both versions? It looks nice on small areas but on large plains (2nd screenshot) it's rather a weird distribution without clusters.

@paramat paramat added the WIP The PR is still being worked on by its author and not ready yet. label May 13, 2018
@paramat
Copy link
Contributor Author

paramat commented May 13, 2018

I made a mistake, retuning. Multiplication would vary blend according to biome.
Will try a more inbetween value too.

@paramat paramat removed the WIP The PR is still being worked on by its author and not ready yet. label May 13, 2018
@paramat paramat changed the title Vertical biome blend: Improve PRNG seed for finer detail Vertical biome blend: Tune PRNG seed for finer detail May 13, 2018
@paramat
Copy link
Contributor Author

paramat commented May 14, 2018

@SmallJoker updated and new screenshots.

@paramat paramat merged commit a01a9ca into minetest:master May 14, 2018
@paramat paramat deleted the impvertblend branch May 19, 2018 22:42
osjc pushed a commit to osjc/minetest that referenced this pull request Jan 23, 2019
luk3yx pushed a commit to luk3yx/minetest that referenced this pull request Mar 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Maintenance Tasks to keep the codebase and related parts in order, including architectural improvements @ Mapgen One approval ✅ ◻️
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants