Skip to content

Conversation

@jselig-rigetti
Copy link
Contributor

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

@windsurf-bot windsurf-bot bot left a comment

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
Copy link

github-actions bot commented Sep 2, 2025

🐰 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
Copy link

github-actions bot commented Sep 2, 2025

☂️ 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🐍

Copy link
Contributor

@asaites asaites left a comment

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

3 participants