Merge pull request argmin-rs#73 from vadixidav/master

remove unecessary Default bound on NelderMead
amfaber · Oct 13, 2020 · 1ac788f · 1ac788f
2 parents 68c282f + 847188f
commit 1ac788f
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/src/solver/neldermead/mod.rs b/src/solver/neldermead/mod.rs
@@ -54,7 +54,7 @@ pub struct NelderMead<P, F> {
 
 impl<P, F> NelderMead<P, F>
 where
-    P: Clone + Default + ArgminAdd<P, P> + ArgminSub<P, P> + ArgminMul<F, P>,
+    P: Clone + ArgminAdd<P, P> + ArgminSub<P, P> + ArgminMul<F, P>,
     F: ArgminFloat,
 {
     /// Constructor
@@ -182,7 +182,7 @@ where
 
 impl<P, F> Default for NelderMead<P, F>
 where
-    P: Clone + Default + ArgminAdd<P, P> + ArgminSub<P, P> + ArgminMul<F, P>,
+    P: Clone + ArgminAdd<P, P> + ArgminSub<P, P> + ArgminMul<F, P>,
     F: ArgminFloat,
 {
     fn default() -> NelderMead<P, F> {
@@ -193,8 +193,7 @@ where
 impl<O, P, F> Solver<O> for NelderMead<P, F>
 where
     O: ArgminOp<Output = F, Param = P, Float = F>,
-    P: Default
-        + Clone
+    P: Clone
         + Serialize
         + DeserializeOwned
         + ArgminScaledSub<O::Param, O::Float, O::Param>