Skip to content

Fix quantum teleportation example to use explicit Pauli gate - #585

Merged
rawkintrevo merged 1 commit into
apache:mainfrom
guan404ming:fix-teleportation-api
Oct 20, 2025
Merged

Fix quantum teleportation example to use explicit Pauli gate#585
rawkintrevo merged 1 commit into
apache:mainfrom
guan404ming:fix-teleportation-api

Conversation

@guan404ming

@guan404ming guan404ming commented Oct 15, 2025

Copy link
Copy Markdown
Member

Purpose of PR

Fix the quantum teleportation example to use the correct method names for Pauli gates. The example was using incorrect method names (apply_x_gate and apply_z_gate) instead of the existing API methods (apply_pauli_x_gate and apply_pauli_z_gate).

Linked Issues

Add links to related issues.

Changes Made

  • Bug fix
  • New feature
  • Documentation update

Important ToDos

Please mark each with an "x"

A GitHub issue exists (if not, please create one) [https://github.com/apache/mahout/issues]

  • Title of PR is "Issue #XXXX: Brief Description of Changes" where XXXX is the GitHub issue number.
  • Created unit tests where appropriate
  • Added correct licenses on newly added files
  • Assigned GitHub issue to self
  • Added documentation in ScalaDocs/JavaDocs and to the website
  • Successfully built and ran all unit tests, verified that all tests pass locally

If all of these items are not yet complete, but you still feel it is appropriate to open a PR, please open it as a Draft PR instead.
Once all requirements are met, you can mark it as ready for review.

Breaking Changes

Does this PR introduce a breaking change?

  • Yes
  • No

Testing & Verification

Describe how you tested the changes.

  • Unit tests added
  • Manually tested

Checklist

  • The title follows the format "MAHOUT-XXXX Brief Description"
  • GitHub issue is created
  • Code follows ASF guidelines

@rawkintrevo

Copy link
Copy Markdown
Contributor

Thanks @guan404ming sorry for slow tern around

@rawkintrevo
rawkintrevo merged commit 20034f8 into apache:main Oct 20, 2025
3 checks passed
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