(LearnerND) Evaluate less circumspheres

it prunes some circumcircles faster it results in: ~20% faster in 2d ~40% faster in 3d

Helps to speed up LearnerND a bit see (#80)

Please share the code you use for benchmarking: we might as well use it combined with asv.

I usually do some testing by running it against some script I wrote to look at the time complexity (PlotComplexity.py)

Then I do however have a question, what is ASV?

Thanks. asv

ah that looks cool, does this work backwards, like, if we add it now, can we clock things of previous releases up to now?

Yes, but we'd have to actually run the benchmarks.

Other than those comments we're good to go.

@anton-akhmerov confirm that merging this is not contingent on your comment about ASV.

@anton-akhmerov confirm that merging this is not contingent on your comment about ASV.

It is indeed not contingent. I mainly wanted to document in the MR discussion how the benchmark is established.

assigned to @Jorn

added 1 commit

• 1e33f914 - remove obsolete condition and add some comments

Compare with previous version

resolved all discussions

should be good to merge now

merged

mentioned in commit 50898f7d