Merge pull request #241 from Walther/push-zvxowquqmkzp

Walther · web-flow · commit 903497468c79 · 2025-11-23T23:03:54.000+02:00
chore: use Unit::new_unchecked in random_cosine_direction()
diff --git a/clovers/src/random.rs b/clovers/src/random.rs
@@ -32,7 +32,7 @@ pub fn random_cosine_direction(rng: &mut SmallRng) -> Direction {
     let y = phi.sin() * r2.sqrt();
 
     let v: Vec3 = Vec3::new(x, y, z);
-    Unit::new_normalize(v)
+    Unit::new_unchecked(v)
 }
 
 /// Internal helper.

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ pub fn random_cosine_direction(rng: &mut SmallRng) -> Direction {`
`32`	`32`	`let y = phi.sin() * r2.sqrt();`
`33`	`33`
`34`	`34`	`let v: Vec3 = Vec3::new(x, y, z);`
`35`		`- Unit::new_normalize(v)`
	`35`	`+ Unit::new_unchecked(v)`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`/// Internal helper.`