This PR adds functionality to COF to allow for the selection of one of two methods: "fast" which will use the old calculation using the full pairwise distance matrix. "memory" which will use a new method, which only calculates distances when needed, thus saving on memory at the price of speed.
This PR follows up on issue #315 (closed)
All Submissions Basics:
-
Have you followed the guidelines in our Contributing document? -
Have you checked to ensure there aren't other open Pull Requests for the same update/change? -
Have you checked all Issues to tie the PR to a specific one?
All Submissions Cores:
-
Have you added an explanation of what your changes do and why you'd like us to include them? -
Have you written new tests for your core changes, as applicable? -
Have you successfully ran tests with your changes locally? -
Does your submission pass tests, including CircleCI, Travis CI, and AppVeyor? -
Does your submission have appropriate code coverage? The cutoff threshold is 95% by Coversall.