Removed debug assert from Quat::new.

Need to rethink this approach.

src/f32/glam_mint.rs

@@ -244,11 +244,18 @@ mod test {
 
     #[test]
     fn test_quaternion() {
-        use crate::{deg, Quat, Vec3};
-        let g = Quat::from_axis_angle(Vec3::new(1.0, 2.0, 3.0).normalize(), deg(270.0));
-        let m = mint::Quaternion::from(g);
+        use crate::Quat;
+        let m = mint::Quaternion {
+            v: mint::Vector3 {
+                x: 1.0,
+                y: 2.0,
+                z: 3.0,
+            },
+            s: 4.0,
+        };
+        let g = Quat::from(m);
+        assert_eq!(g, Quat::from((1.0, 2.0, 3.0, 4.0)));
         assert_eq!(m, g.into());
-        assert_eq!(g, m.into());
     }
 
     #[test]

src/f32/quat_sse2.rs

@@ -21,13 +21,16 @@ pub struct Quat(pub(crate) __m128);
 impl Quat {
     /// Creates a new rotation quaternion.
     ///
-    /// # Preconditions
+    /// This should generally not be called manually unless you know what you are doing. Use one of
+    /// the other constructors instead such as `identity` or `from_axis_angle`.
     ///
-    /// The input must represent a quaternion of unit length.
+    /// `new` is mostly used by unit tests and `serde` deserialization.
     #[inline]
     pub fn new(x: f32, y: f32, z: f32, w: f32) -> Self {
         let q = unsafe { Self(_mm_set_ps(w, z, y, x)) };
-        debug_assert!(q.is_normalized());
+        // might be better as a warning - if this asserts during deserialization it's really
+        // obtuse.
+        // debug_assert!(q.is_normalized());
         q
     }