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

Improve forest generation next to rivers and lakes #68750

Merged
merged 1 commit into from
Oct 20, 2023

Conversation

mqrause
Copy link
Contributor

@mqrause mqrause commented Oct 19, 2023

Summary

Bugfixes "Improve forest generation next to rivers and lakes"

Purpose of change

Fixes #54040

Describe the solution

Simply add rivers and lakes to the forest biome, since that seems to determine how the feathering works.

I should mention that I have no idea of possible side effects of this. At least I didn't see anything obvious while teleporting around.

Describe alternatives you've considered

Testing

Teleported around a bunch and looked at results. Here are a bunch of unnecessarily large, zoomed out images:

Single tile forest island in a lake
New

single_tile_lake

Old

single_tile_lake_old

4 tile forest island in a lake
New

4_tile_lake

Old

4_tile_lake_old

3 tile forest island with cabin and boat
New

3_tile_cabin_boat_lake

Old

3_tile_cabin_boat_lake_old

single tile swamp island in a lake
New

single_tile_swamp_lake

Old

single_tile_swamp_lake_old

small forest island in a river
New (3 tiles)

3_tile_river

Old (1 tile)

single_tile_river_old

River peninsula starting as field with forest at the tip
New (2 field, 1 regular forest)

1_forest_field_peninsula

Old (field to regular forest to thick forest)

thick_forest_field_peninsula_old

normal forest along a river
New

forest_river

Old

forest_river_old

thick forest along a river
New

thick_forest_river

Old

thick_forest_river_old

(I have a few more images, but it's tiring adding them here and it's really just more of the same.)

Additional context

Not sure why island_forest(_thick) has rotations, but it is how it is.

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON <Bugfix> This is a fix for a bug (or closes open issue) astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Oct 19, 2023
@Maleclypse Maleclypse merged commit 9918584 into CleverRaven:master Oct 20, 2023
25 of 26 checks passed
@mqrause mqrause deleted the forest_water_feathering branch October 20, 2023 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Single tile forest islands have too few trees
2 participants