Impl Default for SphericalVec and PolarVec

This sets the r component to 1, which is a more reasonable default.

Make the existing Default for Vector only apply to those with Real basis
to avoid overlapping impls.

Author: jdahlstrom

core/src/math/angle.rs

@@ -477,6 +477,17 @@ impl ZDiv for Angle {}
 // Foreign trait impls
 //
 
+impl<B> Default for SphericalVec<B> {
+    fn default() -> Self {
+        Self::new([1.0, 0.0, 0.0])
+    }
+}
+impl<B> Default for PolarVec<B> {
+    fn default() -> Self {
+        Self::new([1.0, 0.0])
+    }
+}
+
 impl Display for Angle {
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
         let (val, unit) = if f.alternate() {

core/src/math/vec.rs

@@ -505,8 +505,8 @@ impl<Sc: ApproxEq, Sp, const N: usize> ApproxEq<Self, Sc>
 // Foreign trait impls
 //
 
-// Manual impls of Copy, Clone, Eq, and PartialEq to avoid
-// superfluous where S: Trait bound
+// Manual impls of Copy, Clone, Default, Eq, and PartialEq
+// to avoid superfluous where S: Trait bound
 
 impl<R: Copy, S> Copy for Vector<R, S> {}
 
@@ -516,7 +516,7 @@ impl<R: Clone, S> Clone for Vector<R, S> {
     }
 }
 
-impl<R: Default, S> Default for Vector<R, S> {
+impl<R: Default, B, const DIM: usize> Default for Vector<R, Real<DIM, B>> {
     fn default() -> Self {
         Self(R::default(), Pd)
     }