Re: [Scikit-learn-general] Beta / candidate release this weekend

2013-07-26 Thread Joseph Turian
I have pushed all fixes requested by other participants on my pull request. On Fri, Jul 26, 2013 at 3:30 PM, Joseph Turian wrote: > Can I get my feature into this version? > https://github.com/scikit-learn/scikit-learn/pull/2176 > > I was going to push fixes this weekend, to address everyone's

Re: [Scikit-learn-general] Beta / candidate release this weekend

2013-07-26 Thread John Novak
Hi, I've got a branch which is ready for merge; I didn't see it on the list. I think it has been approved: https://github.com/scikit-learn/scikit-learn/pull/1611 If I'm mistaken and it needs some sort of papal blessing, that's cool too. thanks, -John On Jul 25, 2013, at 3:49 AM, Andreas Muel

Re: [Scikit-learn-general] 20 newsgroups classification

2013-07-26 Thread Harold Nguyen
Hi experts, Thank you very much for your quick reply on this. I encountered another error when going through the tutorial noted here: http://scikit-learn.github.io/scikit-learn-tutorial/working_with_text_data.html >>> from sklearn.datasets import fetch_20newsgroups >>> twenty_train = fetch_20new

Re: [Scikit-learn-general] 20 newsgroups classification

2013-07-26 Thread Lars Buitinck
2013/7/27 Vlad Niculae : > Only the current development version, and the upcoming release, has, > as of recently, support for Python 3. Even so, it won't be easy to > support 3.2, we just aim for 3.3 at the moment. > This being said, I have no idea what causes this specific error. That > line seem

Re: [Scikit-learn-general] Beta / candidate release this weekend

2013-07-26 Thread Joseph Turian
Can I get my feature into this version? https://github.com/scikit-learn/scikit-learn/pull/2176 I was going to push fixes this weekend, to address everyone's comments. On Thu, Jul 25, 2013 at 8:40 AM, Lars Buitinck wrote: > 2013/7/25 Jacob Vanderplas : >> On Thu, Jul 25, 2013 at 7:44 AM, Andrea

Re: [Scikit-learn-general] 20 newsgroups classification

2013-07-26 Thread Vlad Niculae
Hi Harold, Only the current development version, and the upcoming release, has, as of recently, support for Python 3. Even so, it won't be easy to support 3.2, we just aim for 3.3 at the moment. This being said, I have no idea what causes this specific error. That line seems unchanged in the curr

Re: [Scikit-learn-general] 20 newsgroups classification

2013-07-26 Thread Harold Nguyen
Hi again, After updating scikit-learn (I think I was running that script from the newest version of scikit-learn, while my installed version was older), I get the following error: $ python document_classification_20newsgroups.py Traceback (most recent call last): File "document_classification_2

[Scikit-learn-general] 20 newsgroups classification

2013-07-26 Thread Harold Nguyen
Hi experts, I was going through the example here: http://scikit-learn.org/stable/auto_examples/document_classification_20newsgroups.html#example-document-classification-20newsgroups-py But when I run, I get the following error message: $ python document_classification_20newsgroups.py --top20 ..

[Scikit-learn-general] New Jenkins bot to launch the tests under windows

2013-07-26 Thread Olivier Grisel
Now there are still many errors / failures (around 40) so I did not put the email notifications yet: https://jenkins.shiningpanda-ci.com/scikit-learn/job/python-2.7-numpy-1.6.1-scipy-0.11.0-windows7-64bit/ Please feel free to send fixes as PRs :) -- Olivier http://twitter.com/ogrisel - http://gi

Re: [Scikit-learn-general] Adding new classifiers

2013-07-26 Thread Olivier Grisel
2013/7/26 Michal Romaniuk : > Working with the debugger, here is what's happening: > > param1 and param2 are both numpy object arrays (containing numerical > arrays). So param1.flat[0] gives an array and param2.flat[0] also gives > an array. And numpy seems to consider equality of two arrays ambigu

Re: [Scikit-learn-general] Adding new classifiers

2013-07-26 Thread Michal Romaniuk
Working with the debugger, here is what's happening: param1 and param2 are both numpy object arrays (containing numerical arrays). So param1.flat[0] gives an array and param2.flat[0] also gives an array. And numpy seems to consider equality of two arrays ambiguous. Michal On 26/07/13 16:26, Andr

Re: [Scikit-learn-general] Adding new classifiers

2013-07-26 Thread Lars Buitinck
2013/7/26 Michal Romaniuk : > I think that this could be because one of the parameters of __init__ is > a numpy array of objects, containing numeric numpy arrays. > > Do you think I could just use Python's copy library in place of clone? The difference between clone and copy is that clone does not

Re: [Scikit-learn-general] Adding new classifiers

2013-07-26 Thread Andreas Mueller
On 07/26/2013 05:15 PM, Michal Romaniuk wrote: > So I followed the advice from Andreas but now I get this error when > calling clone: > > PATH/scikit-learn/sklearn/base.pyc in clone(estimator, safe) > 66 and param1.dtype == param2.dtype > 67 # We

Re: [Scikit-learn-general] Adding new classifiers

2013-07-26 Thread Michal Romaniuk
So I followed the advice from Andreas but now I get this error when calling clone: PATH/scikit-learn/sklearn/base.pyc in clone(estimator, safe) 66 and param1.dtype == param2.dtype 67 # We have to use '.flat' for 2D arrays ---> 68

[Scikit-learn-general] [SPRINT] How to go to the venue tomorrow

2013-07-26 Thread Nelle Varoquaux
Hi everyone, As you may know, we are changing venue tomorrow. tinyclues is hosting the sprint at: 15, rue du Caire 75002 For those of you who are traveling from Télécom, take line 6 to Denfert-Rochereau, and then line 4 to Réaumur-Sebastopol. It's an old parisian building: you'll need to ring at

[Scikit-learn-general] Vagrant / SaltStack configuration to quickly setup a VM to work on scikit-learn.

2013-07-26 Thread Olivier Grisel
I talked about using vagrant to automatically setup a development environment with both python 2 / 3 with other sprint. I pushed my own setup as a github repo with a README.md file to quickly get started if you are interested: https://github.com/ogrisel/my-linux-devbox/ -- Olivier http://twitt

Re: [Scikit-learn-general] Adding new classifiers

2013-07-26 Thread Andreas Mueller
On 07/26/2013 10:01 AM, Lars Buitinck wrote: > 2013/7/26 Andreas Mueller : >> If you are ok with relying on scikit-learn, you can inherit from >> BaseEstimator and ClassifierMixin, then implement fit, predict and >> __init__ (to set the parameters). > We could add some code skeletons to that, like

Re: [Scikit-learn-general] Adding new classifiers

2013-07-26 Thread Lars Buitinck
2013/7/26 Andreas Mueller : > If you are ok with relying on scikit-learn, you can inherit from > BaseEstimator and ClassifierMixin, then implement fit, predict and > __init__ (to set the parameters). We could add some code skeletons to that, like class MajorityClassifier(BaseEstimator, Classifier