search

XanaduAI / thewalrus

A library for the calculation of hafnians, Hermite polynomials and Gaussian boson sampling.
https://the-walrus.readthedocs.io
Apache License 2.0
101 stars 55 forks source link

Adds decompositions #200

Closed nquesada closed 3 years ago

nquesada commented 3 years ago

Adds a new module decompositions with the function williamson in it. This function is necessary for future improvements in the sampling.

nquesada commented 3 years ago

@josh146 I noticed that. Indeed, nothing related to that test has changed and it passes locally...

josh146 commented 3 years ago

I noticed that. Indeed, nothing related to that test has changed and it passes locally...

I think the likely case is that a dependency of The Walrus has recently released a new version, and it's causing something in this function to break? This would explain why it is happening on the CI but not locally.

codecov[bot] commented 3 years ago

Codecov Report

Merging #200 into master will not change coverage. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #200   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           20        21    +1     
  Lines         1131      1164   +33     
=========================================
+ Hits          1131      1164   +33
Impacted Files Coverage Δ
thewalrus/decompositions.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 f779cb9...29af19c. Read the comment docs.