Getting PEPpy on the test cases. That really is nitpicking. Admittedly, I
didn't realise this was prescribed in PEP8 and I am now better for it.
Fixed.


On Wed, Apr 24, 2013 at 8:24 PM, Lars Buitinck <notifications@github.com>wrote:

> In sklearn/metrics/tests/test_metrics.py:
>
> >      y_true = np.array([0, 2, 2, 0])
> >      pred_decision = np.array([-8.5, 0.5, 1.5, -0.3])
> > -    assert_equal(1.2 / 4,
> > -                 hinge_loss(y_true, pred_decision, pos_label=2, neg_label=0))
> > +    with warnings.catch_warnings():
> > +        assert_equal(1.2 / 4, hinge_loss(y_true, pred_decision,
> > +            pos_label=2, neg_label=0))
>
> Nitpick: this should line up with y_true for readability.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/scikit-learn/scikit-learn/pull/1883/files#r3934051>
> .
>