Skip to content

Commit c727090

Browse files
amuelleramueller
committed
stupid python implementation that reproduces secondary method.
Test ties against ground truth in all cases. add reasonably fast cython implementation remove test for not touching output on identical inputs, as we want fit().transform() and fit_transform() to conform. added whatsnew with fixes to isotonic tie handling
1 parent bf1dfdd commit c727090

File tree

5 files changed

+1864
-1136
lines changed

5 files changed

+1864
-1136
lines changed

‎doc/whats_new.rst

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -328,6 +328,9 @@ Bug fixes
328328
:class:`neighbors.NearestNeighbors` return the samples lying on the
329329
boundary for ``algorithm='brute'``. By `Yan Yi`_.
330330

331+
- Fixed handling of ties in :class:`isotonic.IsotonicRegression`.
332+
We now use the weighted average of targets (secondary method). By
333+
`Andreas Müller`_ and `Michael Bommarito <http://bommaritollc.com/>`_.
331334

332335
API changes summary
333336
-------------------

0 commit comments

Comments
 (0)