XanaduAI / strawberryfields

Strawberry Fields is a full-stack Python library for designing, simulating, and optimizing continuous variable (CV) quantum optical circuits.
https://strawberryfields.ai
Apache License 2.0
745 stars 187 forks source link

fix codefactor issues #638

Closed sduquemesa closed 2 years ago

sduquemesa commented 2 years ago

Context: Codefactor is showing 25+ new issues on PR #441

Description of the Change: Applies suggested changes from codefactor

Benefits: Zero codefactor warnings, allows merge of PR #441

Possible Drawbacks: None

Related GitHub Issues: None

codecov[bot] commented 2 years ago

Codecov Report

Merging #638 (d8ac13e) into master (0e06dc6) will decrease coverage by 36.03%. The diff coverage is 36.84%.

@@             Coverage Diff             @@
##           master     #638       +/-   ##
===========================================
- Coverage   98.55%   62.51%   -36.04%     
===========================================
  Files          77       77               
  Lines        8989     8985        -4     
===========================================
- Hits         8859     5617     -3242     
- Misses        130     3368     +3238     
Impacted Files Coverage Δ
strawberryfields/api/devicespec.py 46.87% <0.00%> (-48.44%) :arrow_down:
strawberryfields/decompositions.py 13.89% <0.00%> (-85.88%) :arrow_down:
...trawberryfields/backends/bosonicbackend/backend.py 100.00% <100.00%> (ø)
strawberryfields/backends/states.py 76.81% <100.00%> (-23.03%) :arrow_down:
strawberryfields/backends/tfbackend/__init__.py 60.00% <100.00%> (-13.34%) :arrow_down:
strawberryfields/backends/tfbackend/circuit.py 96.40% <100.00%> (+0.38%) :arrow_up:
strawberryfields/engine.py 70.17% <100.00%> (-26.32%) :arrow_down:
strawberryfields/apps/subgraph.py 10.11% <0.00%> (-89.89%) :arrow_down:
strawberryfields/apps/clique.py 10.67% <0.00%> (-89.33%) :arrow_down:
strawberryfields/apps/qchem/dynamics.py 12.65% <0.00%> (-87.35%) :arrow_down:
... and 44 more

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 0e06dc6...d8ac13e. Read the comment docs.

sduquemesa commented 2 years ago

I'm merging this branch directly with catstate_real used on PR #441 and closing this PR. No need to merge to master.