Skip to content

1849 randomized compiling#1851

Open
erichulburd wants to merge 59 commits into
masterfrom
1849-randomized_compiling
Open

1849 randomized compiling#1851
erichulburd wants to merge 59 commits into
masterfrom
1849-randomized_compiling

Conversation

@erichulburd

@erichulburd erichulburd commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Description

See #1849 . This implement client utilities for randomized compiling through a private pyquil/_qpu module. We can keep this private to indicate that it is an experimental signature and we may make it fully public in a future pyquil v5 release.

Checklist

  • The PR targets the master branch
  • The above description motivates these changes.
  • The change is atomic and can be described by a single commit (your PR will be squashed on merge).
  • All changes to code are covered via unit tests.
  • Parameters and return values have type hints with PEP 484 syntax.
  • Functions and classes have useful Sphinx-style docstrings.
  • (New Feature) The docs have been updated accordingly.
  • (Bugfix) The associated issue is referenced above using auto-close keywords.

@erichulburd erichulburd requested a review from a team as a code owner June 12, 2026 20:33

@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 Jun 12, 2026

Copy link
Copy Markdown

🐰 Bencher Report

Branch1849-randomized_compiling
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.80 s
test/benchmarks/test_program.py::test_instructions📈 view plot
⚠️ NO THRESHOLD
4.15 s
test/benchmarks/test_program.py::test_iteration📈 view plot
⚠️ NO THRESHOLD
4.24 s
🐰 View full continuous benchmarking report in Bencher

Comment thread test/e2e/test_qpu_randomized_compiling.py
Comment thread pyproject.toml Outdated
networkx = ">=2.5"
qcs-sdk-python = ">=0.20.1,<0.22"
quil = ">=0.15.3,<0.18"
quil = "0.18.0rc1"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We'll also want a veritable release for quil 0.18.0.

@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
7993 7078 89% 87% 🟢

New Files

File Coverage Status
pyquil/_qpu/init.py 100% 🟢
pyquil/_qpu/extern_signatures.py 100% 🟢
pyquil/_qpu/randomized_compiling.py 95% 🟢
TOTAL 98% 🟢

Modified Files

File Coverage Status
pyquil/quilbase.py 94% 🟢
TOTAL 94% 🟢

updated for commit: 4a656a6 by action🐍

Comment thread test/e2e/test_qpu_randomized_compiling.py
Comment thread pyquil/_qpu/randomized_compiling.py Outdated
Comment thread test/unit/conftest.py Outdated
@erichulburd erichulburd force-pushed the 1849-randomized_compiling branch from 5f764f1 to 7eee8de Compare June 18, 2026 16:28
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.

1 participant