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

Migrate ncollide docs #2

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

Conversation

Vrixyz
Copy link
Contributor

@Vrixyz Vrixyz commented Oct 16, 2024

Most of https://ncollide.org/ is still relevant for parry.rs ; but needs a few tweaks to be correct ; it would be a great resource for users to learn the library.

I went though and adapted the following parts (checked items still contain limited work to do):

  • geometric representations
    • Adapt composite shapes section with SimdCompositeShape
  • bounding volumes
    • ⚠️ removed DVT section, it should be translated to QBVH.
    • mention utils::obb ?
  • geometric queries
    • absolute coordinates for contacts is not strictly correct 🤔
    • code should be tested (there's a non totally trivial test to adapt)
  • mesh transformations
    • Chapter starts with mentions to ToTriMesh and ToLinestrip traits, and procedural module, which don´t exist in Parry.
    • transition from HACD to V-HACD makes it difficult to adapt some wordings
    • ⚠️ removed example code + reference to path transformations
  • ⚠️ collision detection pipeline, which is within rapier
  • miscellaneous ? ( https://ncollide.org/miscellaneous/ )
    • circumcircle is in Triangle ; it also has a contains_point.
    • sort 🤷
    • triangulate 🤷 (doesn't even exist in ncollide)
  • automation to assess what's changed in a PR Migrate ncollide docs #2 ⚠️ blocking
  • published https://crates.io/crates/strinject

@Vrixyz Vrixyz requested a review from sebcrozet November 6, 2024 15:33
@Vrixyz Vrixyz force-pushed the migrate_ncollide_docs branch 2 times, most recently from 40b9be2 to 04dfb48 Compare November 14, 2024 13:12
@Vrixyz Vrixyz marked this pull request as ready for review November 20, 2024 08:55
@Vrixyz Vrixyz force-pushed the migrate_ncollide_docs branch 2 times, most recently from 871860e to b8ce959 Compare November 21, 2024 14:28
+ grant pull request write permission
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.

1 participant