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.

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

Move axis aliases into one axis enum #386

Open
movabo opened this issue May 5, 2021 · 2 comments · May be fixed by #423
Open

Move axis aliases into one axis enum #386

movabo opened this issue May 5, 2021 · 2 comments · May be fixed by #423
Labels
cat:refactor refactor/clean up/simplifications/etc. feat:octree octree, cube computations prio:low This has low priority.

Comments

@movabo
Copy link
Collaborator

movabo commented May 5, 2021

In the future we'll need multiple constants for each axis.
We should create one enum which holds all the required constants.
@IAmNotHanni IAmNotHanni added the cat:refactor refactor/clean up/simplifications/etc. label May 5, 2021
@movabo movabo mentioned this issue May 5, 2021
Merged
@IceflowRE
Copy link
Member

Also move BBOX_DIRECTIONS and BBOX_CORNERS_ON_FACE_INDICES from the collisions.

@IAmNotHanni IAmNotHanni mentioned this issue May 20, 2021
Merged
@IceflowRE IceflowRE added feat:octree octree, cube computations prio:high This has high priority. labels Sep 29, 2021
@IAmNotHanni
Copy link
Member

Also move BBOX_DIRECTIONS and BBOX_CORNERS_ON_FACE_INDICES from the collisions.

I'm working on that in hanni/collision2

@IceflowRE IceflowRE linked a pull request Dec 3, 2022 that will close this issue
[collision] Second generation of octree collision detection: account for vertex collisions #423
Draft
8 tasks
@IAmNotHanni IAmNotHanni added prio:low This has low priority. and removed prio:high This has high priority. labels May 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
cat:refactor refactor/clean up/simplifications/etc. feat:octree octree, cube computations prio:low This has low priority.
Projects
None yet
Milestone
No milestone
3 participants
@movabo @IAmNotHanni @IceflowRE