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

Set number of qubits for leakage detection #525

Merged
merged 7 commits into from
Oct 25, 2024

Conversation

sjdilkes
Copy link
Contributor

Description

The automatic leakage detection tries to maximise parallelism by using as many spare device qubits as possible for performing leakage detection. Currently when passing leakage_detection=True to process_circuit/s, it automatically assumes that the user wants to make use of all qubits. This PR adds a new kwarg to pytket-quantinuum n_leakage_detection_qubits that allows the user to specify fewer qubits.

Related issues

Please mention any github issues addressed by this PR.

Checklist

  • I have performed a self-review of my code.
  • I have commented hard-to-understand parts of my code.
  • I have made corresponding changes to the public API documentation.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have updated the changelog with any user-facing changes.

@sjdilkes sjdilkes requested a review from yao-cqc as a code owner October 25, 2024 12:59
@yao-cqc
Copy link
Contributor

yao-cqc commented Oct 25, 2024

I think we should update the changelog as well.

@sjdilkes
Copy link
Contributor Author

I think we should update the changelog as well.

Just done!

Copy link
Contributor

@yao-cqc yao-cqc left a comment

Choose a reason for hiding this comment

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

Thanks!

@sjdilkes sjdilkes merged commit b61baf6 into main Oct 25, 2024
12 checks passed
@sjdilkes sjdilkes deleted the set-n-qubits-for-leakage-detection branch October 25, 2024 15:47
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.

2 participants