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

The Great Ocean Revamp #967

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

ikabod-kee
Copy link
Collaborator

@ikabod-kee ikabod-kee commented Jan 26, 2025

Oceans now dominate the sea!
Edit: .... yeah duh

Copy link
Member

@IntegratedQuantum IntegratedQuantum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The climate parameters don't appear well thought out.
The base ocean is antiMountain, the shallow ocean is flatland and the cliffs are mountain
Based on how the climate generator tries to separate these three tags, we end up always seeing cliffs next to shallow ocean. This seems unintentional.

Also why did you remove the islands?

assets/cubyz/biomes/ocean/base.zig.zon Show resolved Hide resolved
assets/cubyz/biomes/ocean/beach.zig.zon Outdated Show resolved Hide resolved
.minHeight = -1,
.maxHeight = 5,
.chance = 0.25,
.mountains = 50,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The cliffs that stick out of the water do appear rather flat/round. Maybe more mountain noise would be good?

@ikabod-kee
Copy link
Collaborator Author

Also why did you remove the islands?
With this new ocean system, islands are automatically generated as individual biomes. I might add them back at some point as little rare biomes.

The base ocean is antiMountain, the shallow ocean is flatland and the cliffs are mountain
Based on how the climate generator tries to separate these three tags, we end up always seeing cliffs next to shallow ocean. This seems unintentional.

Yeah, whoops ^_^

@IntegratedQuantum
Copy link
Member

With this new ocean system, islands are automatically generated as individual biomes. I might add them back at some point as little rare biomes.

Do you have an example of this, because I don't think it really works without subbiomes.

@ikabod-kee
Copy link
Collaborator Author

Do you have an example of this, because I don't think it really works without subbiomes.

image

@ikabod-kee
Copy link
Collaborator Author

Should I convert the ocean cliffs to a sub-biome?

@ikabod-kee
Copy link
Collaborator Author

Right, made a push. I tried to salvage the ideas of cliffs, being sort of like those things pirates would fear. "Arrrg, avoid the Cliffs o' Doom."

I also re-added shelves. There's still work to be done, I just need your opinion.

Copy link
Member

@IntegratedQuantum IntegratedQuantum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think even with the shelf, there are still some cases where the beaches are kind of messy, or falling off too quickly.

.depth = 2,
.smoothness = 0.2,
},
},
.transitionBiomes = .{
.{
.id = "cubyz:beach",
.id = "cubyz:ocean/cliffs",
.chance = 0.25,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The new cliffs look nice, but I think you should make it rarer, the oceans appear to be cluttered with cliffs now.

.id = "cubyz:ocean/cliffs",
.chance = 0.25,
.width = 2,
.properties = .{.antiMountain, .ocean},
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ocean is redundant, since the parent biome is already an ocean anyways.

},
.radius = 500,
.minHeight = -4,
.minHeight = -2,
.maxHeight = 0,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe also set maxHeight to -2 for a more consistent transition?

.chance = 1,
.keepOriginalTerrain = 0.1,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this might break the transition in some cases. But you should test it out if that really is the cause.


.radius = 128,
.minHeight = -64,
.maxHeight = -32,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The shallow biome is kind of difficult to see in the rest of the ocean, maybe it should be shallower?

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