Permalink
Browse files

MISC added note that we need better tests

  • Loading branch information...
1 parent a0498b6 commit e7c4acc5a41e50b6d58f59fcd6dd64b0eedb0dbc @amueller amueller committed Apr 27, 2012
Showing with 4 additions and 0 deletions.
  1. +2 −0 sklearn/manifold/tests/test_isomap.py
  2. +2 −0 sklearn/manifold/tests/test_locally_linear.py
@@ -111,6 +111,8 @@ def test_transform():
def test_pipeline():
# check that Isomap works fine as a transformer in a Pipeline
+ # only checks that no error is raised.
+ # TODO check that it actually does something useful
X, y = datasets.make_blobs(random_state=0)
clf = pipeline.Pipeline(
[('isomap', manifold.Isomap()),
@@ -97,6 +97,8 @@ def test_lle_manifold():
def test_pipeline():
# check that LocallyLinearEmbedding works fine as a Pipeline
+ # only checks that no error is raised.
+ # TODO check that it actually does something useful
from sklearn import pipeline, datasets
X, y = datasets.make_blobs(random_state=0)
clf = pipeline.Pipeline(

0 comments on commit e7c4acc

Please sign in to comment.