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

Reduce lava in biomes-based mapgen #25

Merged
merged 1 commit into from
Jul 12, 2020

Conversation

Treer
Copy link
Collaborator

@Treer Treer commented Jul 5, 2020

I'll merge this myself eventually, so this PR is more of a heads-up and opportunity for anyone to comment, object, or tune.

The biomes-based mapgen creates the same amount of lava ores as the original nether mapgen, but it doesn't have the same issue of chunk emerge order causing lava ores in the overdraw regions to randomly get removed. This adjustment will hopefully balance that difference out a little. (see also paramat's comment about why too much lava isn't necessarily desireable, plus I'm also hoping to add a new region to the nether at some point with lava seas - so people can get their lava-pool fix that way)

Also makes glowstone stalactites a bit rarer.

Adjusting lava ore scarcity in the biomes-based mapgen doesn't cause forwards or backwards compatibilty issues with existing maps, likewise with schematic rarety like glowstone stalactites, so we can afford to fiddle and tune with the biomes-based mapgen.

The biomes-based mapgen was creating the same amount of lava as the original nether mapgen, but it doesn't have the same issue of chunk emerge order sometimes causing lava in the overdraw regions to get removed. This adjustment will hopefully balance that a little.

Also makes glowstone stalactite a bit rarer.

Adjusting lava ore scarcity in the biomes-mapgen doesn't cause forwards or backwards compatibilty issues with existing maps, likewise with schematic rarety like glowstone stalactites, so can afford to fiddle and tune.
@SmallJoker
Copy link
Member

These are not so drastic balance patches. Looks about right 👍

@Treer Treer merged commit 8769593 into minetest-mods:master Jul 12, 2020
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