Closed antalszava closed 4 years ago
Merging #340 into master will not change coverage by
%
. The diff coverage is97.67%
.
@@ Coverage Diff @@
## master #340 +/- ##
=======================================
Coverage 97.70% 97.70%
=======================================
Files 51 51
Lines 6354 6354
=======================================
Hits 6208 6208
Misses 146 146
Impacted Files | Coverage Δ | |
---|---|---|
strawberryfields/api/job.py | 93.02% <ø> (ø) |
|
strawberryfields/io.py | 98.78% <66.66%> (ø) |
|
strawberryfields/api/connection.py | 97.70% <100.00%> (ø) |
|
strawberryfields/apps/sample.py | 100.00% <100.00%> (ø) |
|
strawberryfields/circuitspecs/X12.py | 100.00% <100.00%> (ø) |
|
strawberryfields/circuitspecs/X8.py | 100.00% <100.00%> (ø) |
|
strawberryfields/circuitspecs/__init__.py | 100.00% <100.00%> (ø) |
|
strawberryfields/engine.py | 95.33% <100.00%> (ø) |
|
strawberryfields/program.py | 98.84% <100.00%> (ø) |
|
... and 5 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 2b5a89e...d36295c. Read the comment docs.
Context: The
cli
module contains aping
function to verify the connection to the Xanadu cloud platform. When used in a Python script (and as an option with the command-line interface), anAttributeError
is raised.Description of the Change: Imports
cli.ping
in__init__.py
and adjusts the examples to usesf.ping()
for brevity.Benefits: The
ping
function can be used from within a Python script after importingstrawberryfields
:Possible Drawbacks: Shortening the call to
sf.ping()
in the examples is less descriptive when compared tosf.cli.ping()
.Related GitHub Issues:
338