2013/5/1 Olivier Grisel <olivier.gri...@ensta.org>: > I spend a couple of hours fixing the build infrastructures: > > - the travis CI mldata failures should be fixed by: > > https://github.com/scikit-learn/scikit-learn/commit/0859653e096e7cc48fbdb1e56564482c92efc120 > https://github.com/scikit-learn/scikit-learn/issues/1417 > > - the jenkins build was broken by an upgrade of the shining panda > python versions that broken symlinks of the 2 virtualenv we use on the > debian VM. I rebuilt those from scratch and the build is back up: > > https://jenkins.shiningpanda-ci.com/scikit-learn
Great work! Thanks! > However I observed a large increase in pep8 violations since the last > successful build that occur. > > Please be careful when contributing and merging pull-request: run pep8 > on your code and fix the violations before merging. > > Here is the pep8 violations report: > > https://jenkins.shiningpanda-ci.com/scikit-learn/job/python-2.7-numpy-1.6.2-scipy-0.10.1/violations/ A lot of this is joblib and six, though. Can we turn off pep8 reporting for sklearn/externals? I like to keep at least six in the original state in terms of style so that upgrading it is easier. -- Lars Buitinck Scientific programmer, ILPS University of Amsterdam ------------------------------------------------------------------------------ Introducing AppDynamics Lite, a free troubleshooting tool for Java/.NET Get 100% visibility into your production application - at no cost. Code-level diagnostics for performance bottlenecks with <2% overhead Download for free and get started troubleshooting in minutes. http://p.sf.net/sfu/appdyn_d2d_ap1 _______________________________________________ Scikit-learn-general mailing list Scikit-learn-general@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/scikit-learn-general
