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

WorldCell queries and commands #3092

Closed
wants to merge 5 commits into from

Conversation

Frizi
Copy link
Contributor

@Frizi Frizi commented Nov 9, 2021

A proof of concept implementation for RFC #42.

Implements runtime-checked ECS queries and commands through shared access to WorldCell instance. Those queries see the world as if all queued commands up to their creation point were applied. Every query iterator has its own snapshot of world saved essentially as a diff applied during iteration over the true world data.

This PR is currently not feature-complete, but already useful subset of what would have to be implemented. A few commands are missing (mostly bundles support), and the implementation definitely could be improved in many places. But it does serve a purpose of proving that the proposed RFC design is possible to implement, and should be a good starting point for future work.

@github-actions github-actions bot added the S-Needs-Triage This issue needs to be labelled label Nov 9, 2021
@@ -0,0 +1,457 @@
//! A copy of "appendlist" crate sources, extended with draining iterator.
Copy link
Contributor

Choose a reason for hiding this comment

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

Which version? (crates.io version number or commit hash)

@alice-i-cecile alice-i-cecile added A-ECS Entities, components, systems, and events C-Feature A new feature, making something new possible S-Needs-Review S-Needs-Design-Doc This issue or PR is particularly complex, and needs an approved design doc before it can be merged and removed S-Needs-Triage This issue needs to be labelled S-Needs-Review labels Nov 9, 2021
@alice-i-cecile alice-i-cecile added the X-Controversial There is active debate or serious implications around merging this PR label Apr 25, 2022
@ItsDoot
Copy link
Contributor

ItsDoot commented Jul 15, 2024

WorldCell was removed in #12551, visit the linked PR for the rationale.

@ItsDoot ItsDoot closed this Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-ECS Entities, components, systems, and events C-Feature A new feature, making something new possible S-Needs-Design-Doc This issue or PR is particularly complex, and needs an approved design doc before it can be merged X-Controversial There is active debate or serious implications around merging this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants