Skip to content
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

Update language to "debug" #1485

Merged
merged 4 commits into from
Jun 3, 2024
Merged

Update language to "debug" #1485

merged 4 commits into from
Jun 3, 2024

Conversation

abbycross
Copy link
Collaborator

Closes #925

@abbycross abbycross requested a review from Eric-Arellano June 3, 2024 17:10
@qiskit-bot
Copy link
Contributor

One or more of the following people are relevant to this code:


Quantum simulators can be used to help develop and test programs before fine-tuning them and sending them to quantum hardware. Local simulators can do this with good performance and efficiency.

Because the cost of classically simulating quantum circuits scales exponentially with the number of qubits, circuits that are larger than 50 qubits or so generally cannot be directly verified. For such circuits, you can:
Because the cost of classically simulating quantum circuits scales exponentially with the number of qubits, circuits that are larger than 50 qubits or so generally cannot be directly debugged. For such circuits, you can:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should just explicitly say "cannot be directly run on simulators"

docs/verify/index.mdx Outdated Show resolved Hide resolved
docs/verify/index.mdx Outdated Show resolved Hide resolved
docs/verify/index.mdx Outdated Show resolved Hide resolved
Copy link
Collaborator

@Eric-Arellano Eric-Arellano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

Copy link
Collaborator

@javabster javabster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@abbycross abbycross added this pull request to the merge queue Jun 3, 2024
Merged via the queue into main with commit 5f6da35 Jun 3, 2024
4 checks passed
@abbycross abbycross deleted the ajc/debug-not-verify branch June 3, 2024 18:21
frankharkins pushed a commit to frankharkins/documentation that referenced this pull request Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[ REFACTOR] verify/introduction to talk about "debugging" rather than the "verify" workflow step
4 participants