Skip to content

Add a detect_cyclic_traversal debug feature to bevy_ecs #17465

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.

Sign up for GitHub

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

Jump to bottom
Open
alice-i-cecile opened this issue Jan 21, 2025 · 1 comment
Open

Add a detect_cyclic_traversal debug feature to bevy_ecs #17465

alice-i-cecile opened this issue Jan 21, 2025 · 1 comment
Assignees
@theotherphil
Labels
A-ECS Entities, components, systems, and events C-Usability A targeted quality-of-life change that makes Bevy easier to use D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward

Comments

@alice-i-cecile
Copy link
Member

Follow-up: I reckon a detect_cyclic_traversal debug feature could be nice. Internally just keep an EntityHashSet and panic! when a cycle is detected.

Originally posted by @bushrat011899 in #17463 (comment)
@alice-i-cecile alice-i-cecile added A-ECS Entities, components, systems, and events C-Usability A targeted quality-of-life change that makes Bevy easier to use D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Jan 21, 2025
@theotherphil
Copy link
Contributor

I'll have a go at this one.

@theotherphil theotherphil self-assigned this May 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@theotherphil theotherphil

Labels
A-ECS Entities, components, systems, and events C-Usability A targeted quality-of-life change that makes Bevy easier to use D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@theotherphil @alice-i-cecile