josh146 commented 4 years ago

Context:

In #319, the changes were requested for the hardware tutorial.

Description of the Change:

[x] In hardware tutorial add a download link for test.xbb.

Note: in the refactor, the file test.xbb is no longer used.
[x] Specify the INFILE argument to sf run is a blackbird script
[x] Add thumbnail for the hardware tutorial
[x] In hardware tutorial, link readers back to the operation decorator docstring
[x] Can we restructure this section so that it foremost shows users how to rescale their adjacency matrix (if possible) to give a runnable circuit?

Note: this is non-trivial, so have removed this discussion from the tutorial. I recommend we add in a function that automatically rescales the adjacency matrix for hardware chips.
[x] Also, is the value r=1 an actual physical constraint? I thought it was just binary
[x] thewalrus appears quite suddenly here. Suggest writing a smoother segue.

Note: have removed mention of thewalrus.
[x] Enough to just reproduce the error message, not the full traceback (which anyway contains some unnecessary developer-specific stuff)

Benefits: n/a

Possible Drawbacks: n/a

Related GitHub Issues: #319

codecov[bot] commented 4 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (docs-fix-hardware@e46ad7f). Click here to learn what that means. The diff coverage is n/a.

@@                 Coverage Diff                  @@
##             docs-fix-hardware     #330   +/-   ##
====================================================
  Coverage                     ?   97.66%           
====================================================
  Files                        ?       50           
  Lines                        ?     6252           
  Branches                     ?        0           
====================================================
  Hits                         ?     6106           
  Misses                       ?      146           
  Partials                     ?        0

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 e46ad7f...6c9c0e7. Read the comment docs.

co9olguy commented 4 years ago

Can we add a mailto in a prominant place for people to use when they want to request hardware access. Address will be sf @xanadu.ai

XanaduAI / strawberryfields

Hardware tutorial #330

Codecov Report