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

Stability improvements + remove cuda support #192

Merged
merged 9 commits into from
Apr 28, 2024
Merged

Conversation

sebcrozet
Copy link
Member

This PR removes CUDA support (so we don’t have to stick to an older version of the rust toolchain due to cust being unmaintained) and implements features needed for the upcoming stability improvements in Rapier.

Modified

  • Remove CUDA support to break free from the toolchain restriction required by cust.
  • Rework internal edges resolution using normal cones. This implies the modification of the
    SimdCompositeShape::map_part_at, TypedSimdCompositeShape::map_typed_part, and
    TypedSimdCompositeShape::map_untyped_part trait functions so that the closure argument takes
    an extra argument for the (optional) normal constraints. This argument can be safely ignored
    by user code unless applying the normal collection is relevant to your use-case.
  • Contact manifolds will now retain all contacts (including the ones further than the specified prediction
    distance) whenever any contact is actually closer than this prediction distance.
  • Typo fix: renamed TopologyError::BadAdjascentTrianglesOrientation to BadAdjacentTrianglesOrientation.

Fixed

  • Fix contacts between convex shapes being occasionally ignored due to some rounding errors.
  • Remove crash when entering unreachable code in non-linear TOI calculation.
  • Fix accuracy issue in triangle-mesh center-of-mass calculation when the mesh isn’t manifold.

Added

  • Add SdpMatrix2::inverse_and_get_determinant_unchecked. This is useful for computing the
    inverse in a AoSoA SIMD setting.
  • Add Aabb::intersects_moving_aabb to perform a swept test between two moving aabbs.

@sebcrozet sebcrozet force-pushed the internal-edges-rework branch from d84b560 to fe8f38a Compare April 28, 2024 15:13
@sebcrozet sebcrozet force-pushed the internal-edges-rework branch from 0f036dc to 4ccf84d Compare April 28, 2024 15:43
@sebcrozet sebcrozet merged commit 57a472f into master Apr 28, 2024
5 checks passed
@sebcrozet sebcrozet deleted the internal-edges-rework branch April 28, 2024 15:46
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