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 seeding documentation to indicate what circuit readout processes are seeded #1248

Merged
merged 5 commits into from
Oct 29, 2024

Conversation

paul0403
Copy link
Contributor

Context:
Add a list of seeding-supported circuit readout methods to the documentation.

Benefits:
Avoid potential user misunderstanding when unseeded readout methods are used with a seed.

@paul0403 paul0403 requested review from josh146 and a team October 29, 2024 20:55
Copy link
Contributor

Hello. You may have forgotten to update the changelog!
Please edit doc/releases/changelog-dev.md on your branch with:

  • A one-to-two sentence description of the change. You may include a small working example for new features.
  • A link back to this PR.
  • Your name (or GitHub username) in the contributors section.

Copy link
Contributor

@joeycarter joeycarter 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, thanks! 🚀

@paul0403 paul0403 changed the title update seeding documentation to indicate what circuit readout process… Update seeding documentation to indicate what circuit readout process… Oct 29, 2024
@josh146
Copy link
Member

josh146 commented Oct 29, 2024

@paul0403 to avoid duplicating effort, @joeycarter and I have already updated the docs here :) #1238

@josh146
Copy link
Member

josh146 commented Oct 29, 2024

So this PR may cause a clash (feel free to merge them in a way that is appropriate)

@paul0403
Copy link
Contributor Author

So this PR may cause a clash (feel free to merge them in a way that is appropriate)

Discussed with @joeycarter in person, we will merge this PR first and then resolve a small conflict in the changelog polish PR.

@paul0403 paul0403 changed the title Update seeding documentation to indicate what circuit readout process… Update seeding documentation to indicate what circuit readout processes are seeded Oct 29, 2024
@paul0403 paul0403 added this to the v0.9.0 milestone Oct 29, 2024
@paul0403 paul0403 merged commit d2a3504 into v0.9.0-rc Oct 29, 2024
39 checks passed
@paul0403 paul0403 deleted the document_supported_seeding_readouts branch October 29, 2024 21:34
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.

3 participants