Skip to content

feature: Taller worlds support, WIP.#857

Closed
scrdest wants to merge 3 commits intolouis-e:mainfrom
scrdest:feature/i-have-the-high-ground
Closed

feature: Taller worlds support, WIP.#857
scrdest wants to merge 3 commits intolouis-e:mainfrom
scrdest:feature/i-have-the-high-ground

Conversation

@scrdest
Copy link
Copy Markdown
Contributor

@scrdest scrdest commented Mar 28, 2026

fix: Invalid outliers are no longer throwing off the result due to being counted for min/max height aggregations
feature: Adds a ref to Ground to Landuse/Natural processors to expose height data to the generator logic
refactor: MIN_Y/MAX_Y are now exposed as common crate constants instead of being redefined in multiple places
tweak: Deciduous trees should not generate at mountain altitudes by default
tweak: Default min/max levels are set to 4k heigh worlds (WIP, will likely need a revert)

@scrdest
Copy link
Copy Markdown
Contributor Author

scrdest commented Mar 28, 2026

This is currently a dirty dump of my assorted experiments to generate better mountains in Arnis - the more controversial tweaks like the default height values will be cleaned up (configified or removed).

jmalek added 3 commits April 3, 2026 22:10
fix: Invalid outliers are no longer throwing off the result due to being counted for min/max height aggregations
feature: Adds a ref to Ground to Landuse/Natural processors to expose height data to the generator logic
refactor: MIN_Y/MAX_Y are now exposed as common crate constants instead of being redefined in multiple places
tweak: Deciduous trees should not generate at mountain altitudes by default
tweak: Default min/max levels are set to 4k heigh worlds (WIP, will likely need a revert)
@scrdest scrdest force-pushed the feature/i-have-the-high-ground branch from e752613 to bc65a42 Compare April 3, 2026 21:10
@louis-e
Copy link
Copy Markdown
Owner

louis-e commented Apr 5, 2026

Very cool, thanks! Can't wait to see the finished result :)

@louis-e
Copy link
Copy Markdown
Owner

louis-e commented Apr 6, 2026

I'm quite under time pressure to release a new update, and I promised to deliver an option in the settings for no MAX_Y build height. I'll push a tweak for that on my own, feel free to refactor it if required. Just FYI :)

@scrdest
Copy link
Copy Markdown
Contributor Author

scrdest commented Apr 6, 2026

@louis-e sure thing! The current work around MAX_Y in this branch is frankly quite crude as I initially was just messing around seeing if I can get this working for my own use, so your tweak might be cleaner; I'll work around it one way or another.

I've actually had some sample screenshots ready for like 3 weeks now - I meant to add them to the PR, but got sidetracked (in no small part, by another, far more substantial PR I might have cooking, involving additional integrations... 👀).

There ya go (true scale, Voxy for LODs, Eclipse shaders):

2026-03-15_01 13 10 2026-03-15_01 34 33 2026-03-14_23 50 53

@scrdest
Copy link
Copy Markdown
Contributor Author

scrdest commented Apr 6, 2026

You might know this already, but for MAX_Y, this doesn't work out of the box with stock (Java) Minecraft. The blocks past whatever the build height is set to simply refuse to load in.

The fix is pretty simple - if the save has a datapack ZIP that includes an overworld.json setting {"height": 4064, "min_y": -2032} set, it will load the blocks in with no 'proper' mods required.

Right now I've been fixing this up by copying a datapack into each save manually; in theory we could just generate this programmatically and slap it into savefiles straight from Arnis for less hassle, but it's also a non-obvious feature (and idk how Bedrock handles things).

@louis-e
Copy link
Copy Markdown
Owner

louis-e commented Apr 23, 2026

Thanks a lot, @scrdest!
I picked it up in #976. I made sure to add you as a co-author!

@louis-e louis-e closed this Apr 23, 2026
@scrdest scrdest deleted the feature/i-have-the-high-ground branch April 27, 2026 12:39
@scrdest
Copy link
Copy Markdown
Contributor Author

scrdest commented Apr 27, 2026

Thanks a lot, @scrdest! I picked it up in #976. I made sure to add you as a co-author!

Thank you @louis-e - appreciate the consideration! 😄

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