Should be fixed in current master: https://github.com/scikit-learn/scikit-learn/commit/09677e0193a9dc693f85684bb1c2da56fe6804a2
Although I haven't attempted to reproduce the issue, so please report back if it still not working. Best, Fabian. On Sun, Nov 6, 2011 at 5:56 PM, Olivier Grisel <[email protected]> wrote: > 2011/11/6 Lars Buitinck <[email protected]>: >> >> Shall we apply this workaround in the master as well? That way, grid >> search will also work with user-defined estimators that don't uphold >> the contract; better safe than sorry. I've already refactored Sami's >> script into a testcase with a custom BrokenEstimator. > > Alright but we should not forget to fix the real issue in sparse SVC > and write tests for the "fit" contract as tracked in Mathieu's github > issue. Maybe we should do that during the sprint if nobody does it > before. > > -- > Olivier > http://twitter.com/ogrisel - http://github.com/ogrisel > > ------------------------------------------------------------------------------ > RSA(R) Conference 2012 > Save $700 by Nov 18 > Register now > http://p.sf.net/sfu/rsa-sfdev2dev1 > _______________________________________________ > Scikit-learn-general mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/scikit-learn-general > ------------------------------------------------------------------------------ RSA(R) Conference 2012 Save $700 by Nov 18 Register now http://p.sf.net/sfu/rsa-sfdev2dev1 _______________________________________________ Scikit-learn-general mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/scikit-learn-general
