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

Remove AsyncCollider and AsyncSceneCollider #222

Closed
wants to merge 1 commit into from

Conversation

Shatur
Copy link
Contributor

@Shatur Shatur commented Jul 28, 2022

In this PR I removed the feature I added in #214.

The approach was too limited anyway. You can't set collision groups, friction and other stuff. This will be solved on the Bevy side with asset preprocessing: bevyengine/bevy#4933. In the meanwhile there is a dedicated plugin that can do this and even more: https://github.com/nicopap/bevy-scene-hook.

This will be solved on the Bevy side with asset preprocessing: bevyengine/bevy#4933. In the meanwhile there is a dedicated plugin that can do this and even more: https://github.com/nicopap/bevy-scene-hook.
@Shatur Shatur force-pushed the remove-async-colliders branch from 28e48dc to d159770 Compare July 28, 2022 18:50
@sebcrozet
Copy link
Member

sebcrozet commented Jul 30, 2022

Thank you for this PR!

If we are to deprecate these features, we need to provide a replacement to the user. Having them rely on another crate like bevy-scene-hook sounds like too much of a roundabout where we could provide something more explicitly related to rapier. Also bevy-scene-hook is advertised as just a "proof-of-concept", so I am not comfortable recommending it for production use.

@sebcrozet sebcrozet closed this Jul 30, 2022
@Shatur
Copy link
Contributor Author

Shatur commented Jul 30, 2022

Makes sense!

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