Closed thisac closed 3 years ago
Merging #547 (7248879) into master (33194f6) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #547 +/- ##
=======================================
Coverage 98.22% 98.22%
=======================================
Files 76 76
Lines 8360 8360
=======================================
Hits 8212 8212
Misses 148 148
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 33194f6...7248879. Read the comment docs.
Thanks @josh146 and @antalszava. I've updated the text and heading you commented on. Do you think "Hardware and cloud" works better as a main heading?
it seems like introduction/circuits.rst is also written with having a remote hardware target in mind
I think this part looks fine as it is. It already mentions "remote simulators/hardware" in the "Simulating your program" section, and the other mentions of remote access mainly discusses hardware and hardware compilation, so I think it's fine not mentioning simulators there.
Let me know if you disagree. 🙂
Context: Simulon is now accessible to users directly via
RemoteEngine
but this is not described anywhere in the docs, and thus should be added.Description of the Change:
Benefits: Users will now know that Simulon is accessible as a target for
RemoteEngine
.Possible Drawbacks: None
Related GitHub Issues: None