diff options
| author | Sébastien Crozet <developer@crozet.re> | 2022-06-24 11:56:12 +0200 |
|---|---|---|
| committer | Sébastien Crozet <developer@crozet.re> | 2022-06-24 11:56:12 +0200 |
| commit | 9e431fb392fe85228b695fd5e58c61da7bef1dd2 (patch) | |
| tree | ef4dc0a8744b8d33fe44a8b8bc582bc68fbe3506 | |
| parent | d708ca01d980e7920103ad81feffbf39c6611e5e (diff) | |
| download | rapier-9e431fb392fe85228b695fd5e58c61da7bef1dd2.tar.gz rapier-9e431fb392fe85228b695fd5e58c61da7bef1dd2.tar.bz2 rapier-9e431fb392fe85228b695fd5e58c61da7bef1dd2.zip | |
Address rounding errors resulting in AABBProxy being added to a disjoint region.
| -rw-r--r-- | src/geometry/broad_phase_multi_sap/sap_layer.rs | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/src/geometry/broad_phase_multi_sap/sap_layer.rs b/src/geometry/broad_phase_multi_sap/sap_layer.rs index 216ea05..4316ecd 100644 --- a/src/geometry/broad_phase_multi_sap/sap_layer.rs +++ b/src/geometry/broad_phase_multi_sap/sap_layer.rs @@ -240,6 +240,13 @@ impl SAPLayer { let region_proxy = &mut proxies[region_id]; let region = region_proxy.data.as_region_mut(); + // NOTE: sometimes, rounding errors will generate start/end indices + // that lie outside of the actual region’s AABB. + // TODO: is there a smarter, more efficient way of dealing with this? + if !region_proxy.aabb.intersects(aabb_to_discretize) { + continue; + } + if let Some(actual_aabb) = actual_aabb { // NOTE: if the actual AABB doesn't intersect the // region’s AABB, then we need to delete the |