From 0dc131d1d106f57f26cb12ecee787f3a29a046fd Mon Sep 17 00:00:00 2001 From: Nathaniel Saul Date: Fri, 15 Feb 2019 15:21:58 -0800 Subject: [PATCH] fix broken tests --- kmapper/kmapper.py | 10 ++-------- test/test_coverer.py | 2 +- 2 files changed, 3 insertions(+), 9 deletions(-) diff --git a/kmapper/kmapper.py b/kmapper/kmapper.py index d1e47536..9f5f5210 100644 --- a/kmapper/kmapper.py +++ b/kmapper/kmapper.py @@ -475,17 +475,11 @@ def map( "Deprecation Warning: Please supply km.Cover object. Explicitly passing in n_cubes/nr_cubes and overlap_perc will be deprecated in future releases. ", DeprecationWarning, ) - if coverer is not None: - warnings.warn( - "Deprecation Warning: coverer has been renamed to `cover`. Please use `cover` from now on.", - DeprecationWarning, - ) + # If user supplied nr_cubes, overlap_perc, or coverer, opt for those # TODO: remove this conditional after release in 1.2 - if coverer is not None: - self.cover = coverer - elif nr_cubes is not None or overlap_perc is not None: + if nr_cubes is not None or overlap_perc is not None: n_cubes = nr_cubes if nr_cubes else 10 overlap_perc = overlap_perc if overlap_perc else 0.1 self.cover = Cover(n_cubes=n_cubes, perc_overlap=overlap_perc) diff --git a/test/test_coverer.py b/test/test_coverer.py index 5bb0d52d..c0d2bda5 100644 --- a/test/test_coverer.py +++ b/test/test_coverer.py @@ -82,7 +82,7 @@ def test_complete_pipeline(self, CoverClass): data = data.astype(np.float64) mapper = KeplerMapper() - graph = mapper.map(data, coverer=CoverClass()) + graph = mapper.map(data, cover=CoverClass()) mapper.visualize(graph)