-
Notifications
You must be signed in to change notification settings - Fork 347
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: Update new QPU compiler backend documentation #1708
Conversation
☂️ Python Coverage
Overall Coverage
New FilesNo new covered files... Modified Files
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One suggestion, but otherwise looks good
Co-authored-by: Michael Bryant <[email protected]>
docs/source/introducing_v4.rst
Outdated
|
||
api_options = QPUCompilerAPIOptions() | ||
api_options.use_backend_v2() | ||
api_options.v2( | ||
# See API docs for QPUCompilerAPIOptions for more information on this and other parameters |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we link?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two problems:
-
There isn't a great way to link inside of the code block as far as I can tell, but this isn't such a big deal because it is linked in the description above the code block.
-
Unfortunately, sphinx doesn't forward the docs from the aliased class and it won't render the docstring for a type alias. I've updated the documentation here to link to the type hints for now, but once
qcs-sdk-python
has live docs, we can link to it directly.
Co-authored-by: Kalan <[email protected]>
Description
closes #1707 and #1709