rei-mirrors/Fornjot
1
0
Fork 0
mirror of https://github.com/hannobraun/Fornjot synced 2025-08-14 21:26:06 +00:00
Code Issues Packages Projects Releases Wiki Activity

Prepare for follow-on change

Browse Source
This commit is contained in:
Hanno Braun 2024-10-11 19:29:49 +02:00
parent a0a470009b
commit 35a689cfd7
1 changed files with 8 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
crates/fj-core/src/algorithms/bounding_volume/face.rs
View File

@ -12,6 +12,14 @@ use crate::{
impl super::BoundingVolume<3> for &Face { impl super::BoundingVolume<3> for &Face {
fn aabb(self, geometry: &Geometry) -> Option<Aabb<3>> { fn aabb(self, geometry: &Geometry) -> Option<Aabb<3>> {
// A bounding volume must include the body it bounds, but does not need
// to match it precisely. So it's okay, if it's a bit larger.
//
// Let's just choose a reasonable tolerance value here, then make sure
// we enlarge the AABB accordingly, to make sure it fits.
let tolerance = Tolerance::from_scalar(0.001)
.expect("Tolerance provided is larger than zero");
(self.region().exterior().deref(), self.surface()) (self.region().exterior().deref(), self.surface())
.aabb(geometry) .aabb(geometry)
.map(|aabb2| { .map(|aabb2| {
@ -33,15 +41,6 @@ impl super::BoundingVolume<3> for &Face {
aabb_bottom.merged(&aabb_top) aabb_bottom.merged(&aabb_top)
} }
Path::Line(_) => { Path::Line(_) => {
// A bounding volume must include the body it bounds,
// but does not need to match it precisely. So it's
// okay, if it's a bit larger.
//
// Let's just choose a reasonable tolerance value here,
// then make sure we enlarge the AABB accordingly, to
// make sure it fits.
let tolerance = Tolerance::from_scalar(0.001)
.expect("Tolerance provided is larger than zero");
let offset = Vector::from([tolerance.inner(); 3]); let offset = Vector::from([tolerance.inner(); 3]);
Aabb { Aabb {