Mandrenkov commented 2 years ago

Context: There is a Python installation dependency on repoze.lru which uses a license that may be unsuitable for some projects.

Description of the Change:

Removed repoze.lru as a Python dependency; functools.lru_cache() is now used in place of repoze.lru_cache().

Benefits:

The Walrus has fewer dependencies.

Possible Drawbacks: None.

Related GitHub Issues: None.

nquesada commented 2 years ago

Thanks @Mandrenkov ! This is a most welcome change.

codecov[bot] commented 2 years ago

Codecov Report

Merging #293 (d1b298a) into master (be0339e) will not change coverage. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #293   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           21        21           
  Lines         1446      1446           
=========================================
  Hits          1446      1446

Impacted Files	Coverage Δ
thewalrus/_low_rank_haf.py	`100.00% <100.00%> (ø)`

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update be0339e...d1b298a. Read the comment docs.

XanaduAI / thewalrus

Remove dependency on repoze.lru Python package #293

Codecov Report