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 IgnoredCollisions component #364

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

exoexo-dev
Copy link
Contributor

Objective

Allow to set ignored collisions per entity.

Solution

Added an IgnoredCollisions component which is checked in the broad phase to filter out ignored collisions before any detection checks.


Changelog

Added

Added an IgnoredCollisions component which can be added to entities with a Collider to configure a set of entities with which there will be no collision detection with the owning entity.

@exoexo-dev
Copy link
Contributor Author

exoexo-dev commented May 18, 2024

Looks like something is still missing here - when the entities with ignored collisions are overlapping on spawn they still "explode"

Nvm, it was something else (I was using a joint at a different local anchor from the starting position)

@exoexo-dev exoexo-dev marked this pull request as draft May 18, 2024 12:35
@exoexo-dev exoexo-dev marked this pull request as ready for review May 18, 2024 12:41
@Jondolf Jondolf added C-Enhancement New feature or request A-Collision Relates to the broad phase, narrow phase, colliders, or other collision functionality labels Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Collision Relates to the broad phase, narrow phase, colliders, or other collision functionality C-Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants