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

Rework of the World crate! #69

Merged
merged 21 commits into from
Dec 28, 2024
Merged

Conversation

ghost
Copy link

@ghost ghost commented Nov 24, 2024

This is a significant pull request, so I apologize in advance.

I am currently working on a rewrite of the World crate in BedrockRS to enhance its features and modularity. While this PR is still a work in progress and there are several changes I plan to make, I would greatly appreciate input from the maintainers regarding the code.

Copy link
Member

@theaddonn theaddonn left a comment

Choose a reason for hiding this comment

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

seems very interesting! But I do have some first thoughts...

@ghost ghost marked this pull request as ready for review December 6, 2024 20:28
@ghost
Copy link
Author

ghost commented Dec 6, 2024

I would have liked to implement biomes before i made this PR ready but i have sadly ran low on time and motivation 😅. This is a fully stable implementation to interact with the level. The code is also quite generic; this may lead to generic hell but the library provides powerful default types (at least in my opinion) so users should never have to interact with generic hell!

@ghost ghost requested a review from theaddonn December 20, 2024 21:53
@ghost ghost requested a review from theaddonn December 21, 2024 14:43
@theaddonn
Copy link
Member

@akashic-records-of-the-abyss could you also please switch over to the vek crate for Vec2/3?

@ghost ghost requested a review from theaddonn December 26, 2024 16:38
@theaddonn theaddonn merged commit 306a5f1 into bedrock-crustaceans:dev Dec 28, 2024
4 checks passed
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.

1 participant