Closed thisac closed 2 years ago
Merging #663 (13aade4) into master (e8c1403) will increase coverage by
0.00%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #663 +/- ##
=======================================
Coverage 98.28% 98.28%
=======================================
Files 72 72
Lines 8815 8824 +9
=======================================
+ Hits 8664 8673 +9
Misses 151 151
Impacted Files | Coverage Δ | |
---|---|---|
...trawberryfields/backends/bosonicbackend/backend.py | 100.00% <100.00%> (ø) |
|
strawberryfields/engine.py | 95.91% <100.00%> (-0.02%) |
:arrow_down: |
strawberryfields/ops.py | 98.30% <100.00%> (ø) |
|
strawberryfields/result.py | 94.73% <100.00%> (+1.87%) |
:arrow_up: |
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 e8c1403...13aade4. Read the comment docs.
Gotta love f-strings
:heart:
Context: There's no way to access non-samples results (i.e., metadata) in the
Result
class.Description of the Change:
Result.raw
method is added that returns the raw result dictionary.Result.metadata
method is added that returns the metadata of the results (i.e., everything except the"output"
entry).Result
class.Result.state
allowing a state to be set iff a state has not previously been set and theResult
object containssamples_dict
samples, indicating that it's a local simulation.Benefits:
Result._result
method.Result
class is properly tested.Possible Drawbacks:
samples_dict
(which is currently the case).Related GitHub Issues: None