276
90%

Ran 17 Jul 2018 09:54PM UTC

Jobs 2

Files 67

Run time 13s

Badge

Embed ▾

pending completion

Build # 276

Build Type

push

travis-ci-com

Committed by

web-flow

Commit Message

Fixes Manifold fit_transform bug (#505)

Some manifolds do not provide a transform() method since the data
transformation process cannot be decoupled from the fit() process. In
this case, the user is required to call fit_transform() to get the
transformed data. Unfortunately, the Manifold visualizer implemented the
TransformerMixn and fit() and transform() methods, so when calling the
default fit_transform() method an exception was raised.

Because the fit() method already called fit_transform() - the fix was
simple. Manifold now overrides fit_transform() to do the work that fit()
used to do, and fit() simply calls fit_transform(). In addition, I've
captured AttributeError exceptions in transform() and reraised them with
a message that indicates the user should use fit_transform() instead.

Tests have been added to ensure fit() calls fit_transform(), that
fit_transform() works as expected, and that transform() raises the
correct exception.

Fixes #504

Run Details

2463 of 3061 relevant lines covered (80.46%)

1.61 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	276.1	17 Jul 2018 09:54PM UTC	0	80.46	Travis Job 276.1
2	276.2	17 Jul 2018 09:54PM UTC	0	80.46	Travis Job 276.2

Source Files on build 276

Detailed source file information is not available for this build.

DistrictDataLabs / yellowbrick / 276
90%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 276

DistrictDataLabs / yellowbrick / 276 90%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 276

DistrictDataLabs / yellowbrick / 276
90%

README BADGES
x