Skip to content

fix(docs): clarify what printing a compiled program's output looks like for different targets#1833

Merged
jselig-rigetti merged 2 commits into
masterfrom
1832-docs-clarify-compilation-qvm-vs-qpu
Sep 10, 2025
Merged

fix(docs): clarify what printing a compiled program's output looks like for different targets#1833
jselig-rigetti merged 2 commits into
masterfrom
1832-docs-clarify-compilation-qvm-vs-qpu

Conversation

@jselig-rigetti

Copy link
Copy Markdown
Contributor

@jselig-rigetti jselig-rigetti requested a review from a team as a code owner September 2, 2025 21:00

@windsurf-bot windsurf-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good to me 🤙

💡 To request another review, post a new comment with "/windsurf-review".

@github-actions

github-actions Bot commented Sep 2, 2025

Copy link
Copy Markdown

🐰 Bencher Report

Branch1832-docs-clarify-compilation-qvm-vs-qpu
Testbedci-runner-linux

⚠️ WARNING: No Threshold found!

Without a Threshold, no Alerts will ever be generated.

Click here to create a new Threshold
For more information, see the Threshold documentation.
To only post results if a Threshold exists, set the --ci-only-thresholds flag.

Click to view all benchmark results
BenchmarkLatencyseconds (s)
test/benchmarks/test_program.py::test_copy_everything_except_instructions📈 view plot
⚠️ NO THRESHOLD
10.56 s
test/benchmarks/test_program.py::test_instructions📈 view plot
⚠️ NO THRESHOLD
3.98 s
test/benchmarks/test_program.py::test_iteration📈 view plot
⚠️ NO THRESHOLD
4.02 s
🐰 View full continuous benchmarking report in Bencher

@github-actions

github-actions Bot commented Sep 2, 2025

Copy link
Copy Markdown

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
7303 6419 88% 87% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: 22e05cc by action🐍

@asaites asaites left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jselig-rigetti jselig-rigetti force-pushed the 1832-docs-clarify-compilation-qvm-vs-qpu branch from 3fac711 to 22e05cc Compare September 10, 2025 16:57
@jselig-rigetti jselig-rigetti merged commit b3945f1 into master Sep 10, 2025
20 checks passed
@jselig-rigetti jselig-rigetti deleted the 1832-docs-clarify-compilation-qvm-vs-qpu branch September 10, 2025 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docs: Clarify QVM vs QPU compilation

2 participants