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

Noise settings available in Q# #1997

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Conversation

DmitryVasilevsky
Copy link
Contributor

Added ConfigurePauliNoise and ApplyIdleNoise to Diagnostric namespace to control noise from Q# program.

Copy link
Member

@minestarks minestarks left a comment

Choose a reason for hiding this comment

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

I like the naming.

Shall we consider raising an event (similar to other diagnostic events like Message, DumpMachine, etc) so that the host (i.e. VS Code, Python etc) can inform the user that noise has been applied, if it chooses to?

This could be used to populate accurate info in the histogram view (#1975) or simply just output a message during execution like "noise was applied".

@DmitryVasilevsky DmitryVasilevsky marked this pull request as ready for review November 5, 2024 21:45
Copy link

github-actions bot commented Nov 5, 2024

Change in memory usage detected by benchmark.

Memory Report for 628b84d

Test This Branch On Main Difference
compile core + standard lib 18005745 bytes 17964197 bytes 41548 bytes

Copy link

github-actions bot commented Nov 7, 2024

Change in memory usage detected by benchmark.

Memory Report for 1b45dd2

Test This Branch On Main Difference
compile core + standard lib 18005745 bytes 17964197 bytes 41548 bytes

Copy link

github-actions bot commented Nov 7, 2024

Change in memory usage detected by benchmark.

Memory Report for 61068b4

Test This Branch On Main Difference
compile core + standard lib 18005745 bytes 17964197 bytes 41548 bytes

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