+1 for backward compatibility!


On Tue, May 27, 2014 at 1:36 PM, Satrajit Ghosh <notifications@github.com>wrote:

> In nipype/algorithms/misc.py:
>
> > @@ -28,6 +28,10 @@
> >
> >  from .. import logging
> >
> > +from .. import warnings
> > +warnings.warn('Evaluation interfaces (Distance, Overlap, FuzzyOverlap)
> > +               have been moved to nipype.algorithms.eval' )
>
> also could we maintain backward compatibility for one release (@chrisfilo<https://github.com/chrisfilo>this should be 0.10 not 1.0 yet) and then deprecate the following release.
> this warning should change to a DeprecationWarning and import these
> interfaces from .eval import ...
>
> if we really wanted to give the warning only when someone imports from
> misc, we would have to add dummy classes, something like:
>
> (also could rename eval to metrics?)
>
> import nipype.algorithms.metrics as nam
>
> class Distance (nam.Distance):
>     DeprecationWarning(...)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/nipy/nipype/pull/830/files#r13072770>
> .
>