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

feat: supports collision filtering #74

Merged
merged 7 commits into from
May 24, 2024
Merged

feat: supports collision filtering #74

merged 7 commits into from
May 24, 2024

Conversation

noih
Copy link
Contributor

@noih noih commented Apr 20, 2024

I implemented a group method similar to box2d collision filtering that uses only bit operations, ensuring minimal performance impact. The stress demo has also been updated for observing the results.

I'll be using this functionality but unsure if others will also need it ~

@noih
Copy link
Contributor Author

noih commented Apr 21, 2024

I didn't include dist / docs in the commit due to potential conflicts with multiple PRs, but I'm unsure if it's more convenient for you.

…callback

feat: the raycast `allow` callback adds the `ray` parameter for 'checkInteract' function, placed second for compatibility
@Prozi
Copy link
Owner

Prozi commented May 24, 2024

this is quite interesting and I mostly like it

I will clone adjust and merge

see this in a few days @noih !!

thank you for pull request !!

@Prozi
Copy link
Owner

Prozi commented May 24, 2024

awesome MR I've read all

I plan to move:

  Box: dec("0000 0000 0000 0001"),
  Circle: dec("0000 0000 0000 0010"),
  Ellipse: dec("0000 0000 0000 0100"),
  Line: dec("0000 0000 0000 1000"),
  Polygon: dec("0000 0000 0001 0000"),

to some export in lib

and reuse it

then build dist and docs

thanks @noih !!

@Prozi
Copy link
Owner

Prozi commented May 24, 2024

also

"I'll be using this functionality but unsure if others will also need it" was basically why I started making opensource

@Prozi
Copy link
Owner

Prozi commented May 24, 2024

actually I don't see any breakin changes here SO YOLO I MERGE AND THEN UPDATE

@Prozi Prozi merged commit 4625dfd into Prozi:master May 24, 2024
1 of 3 checks passed
@Prozi
Copy link
Owner

Prozi commented May 24, 2024

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.

None yet

2 participants