Skip to content

Fix neutral atom simulation mutating caller's NoiseConfig - #4218

Open
Anshika Grover (anshikagrover15) wants to merge 1 commit into
microsoft:mainfrom
anshikagrover15:fix/noise-config-mutation
Open

Anshika Grover (anshikagrover15) wants to merge 1 commit into
microsoft:mainfrom
anshikagrover15:fix/noise-config-mutation

Conversation

@anshikagrover15

Copy link
Copy Markdown

Summary

Closes #4094.

  • Add NoiseConfig.copy() to create an independent copy of gate-noise tables.
  • Shallow-copy intrinsics to avoid duplicating the potentially large table.
  • Update NeutralAtomDevice.simulate() to apply RZ noise inheritance to a copy instead of modifying the caller's configuration.
  • Add a regression test verifying caller configuration remains unchanged and resetting RZ noise restores noiseless simulation.

Testing

  • Focused regression test: passed.
  • Python syntax validation: passed.
  • git diff --check: passed.

@anshikagrover15

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

This branch has not been deployed

No deployments
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.

[Bug] Neutral-atom simulation modifies the caller's NoiseConfig

1 participant