Update TEM documentation #2088
Update TEM documentation #2088
Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
|
Thanks for contributing to Qiskit documentation! Before your PR can be merged, it will first need to pass continuous integration tests and be reviewed. Sometimes the review process can be slow, so please be patient. Thanks! 🙌 One or more of the following people are relevant to this code: |
|
|
There was a problem hiding this comment.
Hey @ludmilaasb, thank you and the team for these updates. I've added a few minor requests
Co-authored-by: abbycross <[email protected]>
Co-authored-by: abbycross <[email protected]>
There was a problem hiding this comment.
Tiny tweaks in one sentence - looks good to me otherwise, but @pandasa123 should give the final approval. Thanks!
Co-authored-by: abbycross <[email protected]>
docs/guides/algorithmiq-tem.ipynb
Outdated
| "metadata": {}, | ||
| "outputs": [], | ||
| "source": [ | ||
| "import logging\n", | ||
| "result = job.result()" |
There was a problem hiding this comment.
One code example should show how to extract the expectation value, e.g. result[0].data.evs. It can be this one, or the one in output, or both.
There was a problem hiding this comment.
All right, I added the suggested snippet to the notebook cell
| "source": [ | ||
| "## Inputs\n", | ||
| "\n", | ||
| "**Parameters**\n", |
There was a problem hiding this comment.
It bothers me slightly that the parameters are not all in the same order across functions, even though they are all the same. But it's not a huge deal.
There was a problem hiding this comment.
I changed the order to the following:
- pubs
- backend_name
- instance
- options
No description provided.