On Sat, Jul 27, 2013 at 07:38:43AM -0700, Andreas Mueller wrote:
> In sklearn/preprocessing/imputation.py:

> > @@ -295,6 +295,12 @@ def _dense_fit(self, X, strategy, missing_values, axis):

> >          # Median
> >          elif strategy == "median":
> > +            if np.__version__.split('.')[:2] < (1, 5):

> this is not the best way to do this. there is distutils.version.StrictVersion
> but I'm not sure we want to import that here ;)

That's what I thought. The benefits of distutils.version is to be robust
to dev versions, but we don't need this here.