Closed heltluke closed 2 years ago
Merging #326 (6caaee5) into master (e55b508) will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## master #326 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 22 23 +1
Lines 1616 1645 +29
=========================================
+ Hits 1616 1645 +29
Impacted Files | Coverage Δ | |
---|---|---|
thewalrus/quantum/distinguishable_squeezers.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 e55b508...6caaee5. Read the comment docs.
@heltluke Don't forget to also add yourself (and/or the person who created the functions) to the contributors section in the changelog.
Incidentally, I note that your pull_request_template.md refers to pylint pennylane/path/to/file.py.
Good catch! 😄👍
Don't forget to also add yourself (and/or the person who created the functions) to the contributors section in the changelog.
Whoops. Merged before I saw this. Also not sure who wrote them initially. Likely @nquesada or @jakeffbulmer.
I think this is @nquesada's code
Context: Calculations involving distinguishable squeezers can serve as a starting point to compare experiments making use of ostensibly distinguishable squeezers against.
Description of the Change: Adds three methods for calculating output of distinguishable squeezers incident on an interferometer.
Benefits: Can now make calculations of the output of distinguishable squeezers incident on an interferometer.
Possible Drawbacks: N/A
Related GitHub Issues: N/A
Incidentally, I note that your
pull_request_template.md
refers topylint pennylane/path/to/file.py
.