`clean_by_endpoints` should be able to return the indices. #204

arokem · 2020-01-16T04:21:03Z

For now, just the failing unit test. Implementation to follow.

pep8speaks · 2020-01-23T22:42:16Z

Hello @arokem! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Also removes some unused imports.

arokem · 2020-02-07T00:11:22Z

OK. This is now ready to go. Once merged and integrated, I will push out a new release.

arokem force-pushed the clean_endpoint_return_idx branch from bb8f10c to ee5cfed Compare January 23, 2020 21:55

arokem added 4 commits January 28, 2020 10:16

TST: clean_by_endpoint should be able to return the indices.

482142b

RF: Enables returning the indices

b7fc578

Implement returning of indices from within the clean_by_endpoints block.

c90a958

PEP8

0741e1a

arokem force-pushed the clean_endpoint_return_idx branch from d7e31eb to 0741e1a Compare January 28, 2020 18:17

arokem added 2 commits January 28, 2020 14:31

Return correct indices when selecting by endpoints.

75404f4

BF: Use the new variable.

a3450fc

Also removes some unused imports.

arokem merged commit 6559a80 into yeatmanlab:master Feb 7, 2020

Provide feedback