Optimizations

[kckennylau]

No description provided.

[Mego]

Make the change on line 731 like I mentioned in the comment, and this should be good to merge, assuming the CI build succeeds.

[codecov-io]

Codecov Report

Merging #83 into master will decrease coverage by 0.16%.
The diff coverage is 67.34%.

@@            Coverage Diff             @@
##           master      #83      +/-   ##
==========================================
- Coverage   87.19%   87.02%   -0.17%     
==========================================
  Files           7        7              
  Lines        1616     1619       +3     
  Branches      314      295      -19     
==========================================
  Hits         1409     1409              
- Misses        151      154       +3     
  Partials       56       56

[kckennylau]

I've changed from copy import copy as _copy to from copy import deepcopy as _copy.

[Mego]

This wasn't necessary - copy in the global scope is a wrapper around copy.copy (well, now, copy.deepcopy) that properly handles generators. It should be tmp = [copy(b) for _ in range(a)] + tmp.

[kckennylau]

I just noticed that you have an as_list function that you wrote yourself. Should I replace all occurrences of list with as_list?

[kckennylau]

Actually, I shouldn't. as_list wraps non-iterables.

[Mego]

No, as_list wraps singletons, and deeply converts iterables to lists. It's very expensive, and only really needed when working with generators that can yield other generators.