Skip to content

Conversation

7Rocky
Copy link
Contributor

@7Rocky 7Rocky commented May 22, 2025

Summary

Fixed AND terms for OR.

Issue: Qiskit/qiskit#14434

Fixed AND terms for OR
@7Rocky 7Rocky requested a review from a team as a code owner May 22, 2025 15:04
@qiskit-bot qiskit-bot added the Community PR PRs from contributors that are not 'members' of the Qiskit repo label May 22, 2025
@qiskit-bot
Copy link
Collaborator

Thank you for opening a new pull request.

Before your PR can be merged it will first need to pass continuous integration tests and be reviewed. Sometimes the review process can be slow, so please be patient.

While you're waiting, please feel free to review other open PRs. While only a subset of people are authorized to approve pull requests for merging, everyone is encouraged to review open pull requests. Doing reviews helps reduce the burden on the core team and helps make the project's code better for everyone.

One or more of the following people are relevant to this code:

  • @Cryoris
  • @Qiskit/terra-core
  • @ajavadia

@CLAassistant
Copy link

CLAassistant commented May 22, 2025

CLA assistant check
All committers have signed the CLA.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@ShellyGarion ShellyGarion linked an issue May 22, 2025 that may be closed by this pull request
@alexanderivrii
Copy link
Member

Thanks for doing this! You need to sign CLA (Contributor License Agreement). Also, could you add a short release note (calling an OrGate as "and" could have possibly led to various problems).

….yaml

Co-authored-by: Julien Gacon <gaconju@gmail.com>
@Cryoris Cryoris added stable backport potential The bug might be minimal and/or import enough to be port to stable Changelog: Bugfix Include in the "Fixed" section of the changelog labels May 23, 2025
@coveralls
Copy link

coveralls commented May 23, 2025

Pull Request Test Coverage Report for Build 15234826798

Details

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage increased (+0.007%) to 88.34%

Files with Coverage Reduction New Missed Lines %
crates/transpiler/src/passes/unitary_synthesis.rs 1 94.85%
Totals Coverage Status
Change from base Build 15216054136: 0.007%
Covered Lines: 78480
Relevant Lines: 88839

💛 - Coveralls

@alexanderivrii alexanderivrii added this pull request to the merge queue May 25, 2025
Merged via the queue into Qiskit:main with commit c61d797 May 25, 2025
26 checks passed
mergify bot pushed a commit that referenced this pull request May 25, 2025
* Update quantum_or.py

Fixed AND terms for OR

* Create fix-calling-OR-gate-as-AND-f1b364d04734098d.yaml

* Update releasenotes/notes/fix-calling-OR-gate-as-AND-f1b364d04734098d.yaml

Co-authored-by: Julien Gacon <gaconju@gmail.com>

---------

Co-authored-by: Julien Gacon <gaconju@gmail.com>
Co-authored-by: Alexander Ivrii <alexi@il.ibm.com>
(cherry picked from commit c61d797)
github-merge-queue bot pushed a commit that referenced this pull request May 25, 2025
* Update quantum_or.py

Fixed AND terms for OR

* Create fix-calling-OR-gate-as-AND-f1b364d04734098d.yaml

* Update releasenotes/notes/fix-calling-OR-gate-as-AND-f1b364d04734098d.yaml



---------



(cherry picked from commit c61d797)

Co-authored-by: Rocky <rockygitt@gmail.com>
Co-authored-by: Julien Gacon <gaconju@gmail.com>
Co-authored-by: Alexander Ivrii <alexi@il.ibm.com>
@mtreinish
Copy link
Member

@Mergifyio backport stable/1.4

Copy link
Contributor

mergify bot commented May 27, 2025

backport stable/1.4

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request May 27, 2025
* Update quantum_or.py

Fixed AND terms for OR

* Create fix-calling-OR-gate-as-AND-f1b364d04734098d.yaml

* Update releasenotes/notes/fix-calling-OR-gate-as-AND-f1b364d04734098d.yaml

Co-authored-by: Julien Gacon <gaconju@gmail.com>

---------

Co-authored-by: Julien Gacon <gaconju@gmail.com>
Co-authored-by: Alexander Ivrii <alexi@il.ibm.com>
(cherry picked from commit c61d797)
github-merge-queue bot pushed a commit that referenced this pull request May 28, 2025
* Fixed AND terms for OR in `quantum_or.py` (#14436)

* Update quantum_or.py

Fixed AND terms for OR

* Create fix-calling-OR-gate-as-AND-f1b364d04734098d.yaml

* Update releasenotes/notes/fix-calling-OR-gate-as-AND-f1b364d04734098d.yaml

Co-authored-by: Julien Gacon <gaconju@gmail.com>

---------

Co-authored-by: Julien Gacon <gaconju@gmail.com>
Co-authored-by: Alexander Ivrii <alexi@il.ibm.com>
(cherry picked from commit c61d797)

* Update release note

---------

Co-authored-by: Rocky <rockygitt@gmail.com>
Co-authored-by: Julien Gacon <gaconju@gmail.com>
Co-authored-by: Alexander Ivrii <alexi@il.ibm.com>
Co-authored-by: Matthew Treinish <mtreinish@kortar.org>
rahaman-quantum pushed a commit to rahaman-quantum/qiskit that referenced this pull request Jun 20, 2025
* Update quantum_or.py

Fixed AND terms for OR

* Create fix-calling-OR-gate-as-AND-f1b364d04734098d.yaml

* Update releasenotes/notes/fix-calling-OR-gate-as-AND-f1b364d04734098d.yaml

Co-authored-by: Julien Gacon <gaconju@gmail.com>

---------

Co-authored-by: Julien Gacon <gaconju@gmail.com>
Co-authored-by: Alexander Ivrii <alexi@il.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: Bugfix Include in the "Fixed" section of the changelog Community PR PRs from contributors that are not 'members' of the Qiskit repo stable backport potential The bug might be minimal and/or import enough to be port to stable
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Fix name to "or"
7 participants