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

Add multi noise biome source #445

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

Conversation

urisinger
Copy link
Contributor

@urisinger urisinger commented Jan 2, 2025

Description

this pr adds MultiNoiseBiomeSource and BiomeEntries to generate biomes, which is supposed to mimic MultiNoiseUtil and MultiNoiseBiomeSource.

Testing

not yet

Please follow our Coding Guidelines

}

pub struct MultiNoiseSampler {
pub(crate) temperature: Box<dyn ComponentReference<NoEnvironment>>,
Copy link
Contributor

Choose a reason for hiding this comment

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

This can be a shared or mutable component reference depending on what is needed, if no environment is needed, they can probably be SharedComponentReference and then you can call sample instead of sample_mut

@@ -59,3 +59,4 @@ uuid = { version = "1.11.0", features = ["serde", "v3", "v4"] }
derive_more = { version = "1.0.0", features = ["full"] }
serde = { version = "1.0", features = ["derive"] }
serde_json = "1.0"
serde_with = "3.12"
Copy link
Member

Choose a reason for hiding this comment

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

Why this was added ?, And is there a way to get the same done without this ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it was added to serialize {"key":value} as Vec<(Key,Value)>, i can change the format of the data in extractor to be an array

Copy link
Member

Choose a reason for hiding this comment

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

Having a library less seems better

@Snowiiii
Copy link
Member

Snowiiii commented Jan 6, 2025

@urisinger any progress on this one ?

@urisinger
Copy link
Contributor Author

@urisinger any progress on this one ?

I've been busy the past few days, I'll pick it up next week

@Q2297045667
Copy link

I think it might be time to officially update it now?I'm really looking forward to the implementation of this feature.

我觉得或许现在正是更新它的时候?我非常期待这个功能的实现。

@urisinger
Copy link
Contributor Author

urisinger commented Jan 13, 2025

Sorry for the delay, I'll start working on it again today, I have some changes regarding the general biome generator to finish then I can merge the full biome list that was added and implement it. Then I'll start working on generating each biome in a separate pr

@Q2297045667
Copy link

Sorry for the delay, I'll start working on it again today, I have some changes regarding the general biome generator to finish then I can merge the full biome list that was added and implement it. Then I'll start working on generating each biome in a separate pr

Oh wow,this could be a great PR.

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.

4 participants