-
Notifications
You must be signed in to change notification settings - Fork 12
Feat new gate #191
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
Open
MathieuG-Colibri
wants to merge
122
commits into
dev
Choose a base branch
from
feat-New-gate
base: dev
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Feat new gate #191
Changes from all commits
Commits
Show all changes
122 commits
Select commit
Hold shift + click to select a range
309f3eb
feat: add Rzz gate implementation and update initialization parameters
JulienCalistoTD ba3648b
fix: update CustomGate to use identity operation and improve label fo…
JulienCalistoTD 8bf4811
feat: add PRX gate implementation and update gate imports
JulienCalistoTD 0ee55c9
feat: update PRX gate to use parameters directly and improve language…
JulienCalistoTD c1852e4
feat: update PRX gate to use sympy functions for trigonometric calcul…
JulienCalistoTD 7495a58
feat: refactor PRX gate to use Rz and Rx matrices for canonical matri…
JulienCalistoTD f9219fc
feat: update PRX gate initialization to store target as a list
JulienCalistoTD 0d796e2
feat: update PRX gate to use 'targets' attribute for multiple target …
JulienCalistoTD 1201814
feat: update PRX gate to return RZ and RX gates for Qiskit integration
JulienCalistoTD 5f4f658
chore: Files formated
github-actions[bot] c9bf744
feat: add disable_qubit_rewiring option to device.run calls in run_br…
JulienCalistoTD ede5deb
Merge branch 'feat-New-gate' of https://github.com/ColibrITD-SAS/mpqp…
JulienCalistoTD 68e333f
chore: Files formated
github-actions[bot] e29381a
feat: adjust target indexing for Braket circuit instructions in QCircuit
JulienCalistoTD 2d28df5
Merge branch 'feat-New-gate' of https://github.com/ColibrITD-SAS/mpqp…
JulienCalistoTD f9d71cb
feat: update Braket circuit handling to use add_verbatim_box method
JulienCalistoTD 4331396
feat: optimize target indexing for Braket circuit instructions in QCi…
JulienCalistoTD 76af3b6
feat: update Braket circuit handling to directly add instructions wit…
JulienCalistoTD 05bdd6d
feat: update installation instructions for providers in getting-start…
JulienCalistoTD 9017258
fix: PRX repr and Braket translation issues
MathieuG-Colibri de7a220
feat: fix adjust_measure on simple pauli strings
MathieuG-Colibri c2001a0
fix: adjust_measure padding, apply pre_measure swap
MoHermes 2d4eb43
fix: remove pre_measure swaps, apply direct embedding
MoHermes 9340557
chore: Files formated
github-actions[bot] 724f062
fix: added observables all providers handling
MathieuG-Colibri e969527
chore: Files formated
github-actions[bot] 12a4bb0
Merge branch 'dev' into feat-New-gate
MathieuG-Colibri 8ebd474
feat: added ComposedGates and fixed tests
MathieuG-Colibri 1050804
Merge branch 'feat-New-gate' of https://github.com/ColibrITD-SAS/mpqp…
MathieuG-Colibri 939f92e
chore: Files formated
github-actions[bot] 8a30c81
feat: added testsuite
MathieuG-Colibri 442132e
fix: merge and testsuite
MathieuG-Colibri 60ef840
chore: Files formated
github-actions[bot] bdbb6b7
fix: pyright
MathieuG-Colibri 6343ce4
Merge branch 'feat-New-gate' of github.com:ColibrITD-SAS/mpqp into fe…
MathieuG-Colibri 56f8508
feat: added test and translation for ccu
MathieuG-Colibri 7571814
fix: ccu qiskit and braket issues, changed cirq unitary behavior to h…
MathieuG-Colibri aef067c
chore: Files formated
github-actions[bot] 001d400
fix: custom gate handling in cirq, decomposition bugs, particular gat…
MathieuG-Colibri aae68c2
fix: unitary decomposition non ordered qubits
MathieuG-Colibri 0bd147c
feat: add native Rxx and Ryy rotation gates
MoHermes 18d6c9c
fix: stabilize CustomGate translation across qiskit and cirq, tests p…
MoHermes 9b6e0a2
chore: Files formated
github-actions[bot] a6d2be7
fix: type annotations for decomposition angles
MoHermes 25e8f71
Merge branch 'feat-New-gate' of github.com:ColibrITD-SAS/mpqp into fe…
MoHermes 1d297cf
test: clarify CustomControlledGate translation tests
MoHermes 0a971f5
test: add Rxx and Ryy composed decompositions
MoHermes 7864f92
docs: add Rxx and Ryy documentation
MoHermes c87d5c4
test: cover Rxx, Ryy, Rzz and PRX gates
MoHermes f4477b3
docs: update docs
MoHermes bfe1331
chore: kron simplification
Henri-ColibrITD 0cb5086
Merge branch 'feat-New-gate' of github.com:ColibrITD-SAS/mpqp into fe…
Henri-ColibrITD 9cd00e3
fix: merged branches
MathieuG-Colibri bd7e836
fix: handle composed gates in mpqp_to_qiskit function
JulienCalistoTD 24a002d
fix: gate sets for gate decomposition
ThomasB-Colibri be2818a
chore: Files formated
github-actions[bot] 67ad59a
fix: set definitions
ThomasB-Colibri c03e2f6
fix: merge conflict
ThomasB-Colibri 4ba2e45
chore: Files formated
github-actions[bot] 196b495
fix: merge conflict
ThomasB-Colibri dfff38f
chore: Files formated
github-actions[bot] 57b4c1a
fix: cirq set gates
ThomasB-Colibri 0eea2e2
chore: Files formated
github-actions[bot] 8392224
feat: devices reorganization
ThomasB-Colibri 24dd8da
chore: Files formated
github-actions[bot] d84863c
fix: qasm documentation issue
ThomasB-Colibri 997438d
Merge branch 'feat-new-gate-improvements' of github.com:ColibrITD-SAS…
ThomasB-Colibri aa3ae6c
fix: qasm test
ThomasB-Colibri 1c36562
chore: Files formated
github-actions[bot] 310d2f1
fix: custom gate type
ThomasB-Colibri c8e7f12
fix: wrong typing in gate
ThomasB-Colibri 3af7b96
chore: Files formated
github-actions[bot] 54ab3d9
chore: UnsopportedGateError in error file
ThomasB-Colibri c30eec9
chore: Files formated
github-actions[bot] 1190406
chore: doctring in gate decomposition
ThomasB-Colibri 2f03b2c
chore: Files formated
github-actions[bot] 0fd0145
Merge pull request #213 from ColibrITD-SAS/feat-new-gate-improvements
ThomasB-Colibri db1bd01
fix: repr of Rzz, Rxx, Ryy & PRx
ThomasB-Colibri 36d416a
chore: Files formated
github-actions[bot] 7adbea6
merge: dev in new gate
ThomasB-Colibri feea4b4
Merge branch 'feat-New-gate' of github.com:ColibrITD-SAS/mpqp into fe…
ThomasB-Colibri 57a5330
fix: merge conflicts
ThomasB-Colibri 340a0e6
chore: Files formated
github-actions[bot] 4f78d31
fix: merge conflicts with provider translation
ThomasB-Colibri 6edd6cc
chore: Files formated
github-actions[bot] 6878de5
fix: print of controls in custom controlled gate
ThomasB-Colibri 342cb8b
fix: remove trashfiles
ThomasB-Colibri f92c756
chore: Files formated
github-actions[bot] b40c30d
chore: custom controlled gate spacing
ThomasB-Colibri e4cd491
Merge branch 'feat-New-gate' of github.com:ColibrITD-SAS/mpqp into fe…
ThomasB-Colibri 6b7ceee
fix: defined gates used in PRX
ThomasB-Colibri e64b6e9
chore: ajustments on device
ThomasB-Colibri 29a2441
chore: Files formated
github-actions[bot] 04a93ec
fix: remove wrong test
ThomasB-Colibri 9e338cb
fix: handling of warnings & typing in native gates
ThomasB-Colibri a001c4f
chore: Files formated
github-actions[bot] 369c009
fix: typing report reupdated
ThomasB-Colibri 7edab45
chore: Files formated
github-actions[bot] 0f90a91
chore: helpers in tools
ThomasB-Colibri 490ba7a
chore: Files formated
github-actions[bot] 28abea6
Merge pull request #216 from ColibrITD-SAS/fix-native-gates-typing
ThomasB-Colibri ca6e95f
fix: doc device
MathieuG-Colibri 72669de
Merge branch 'dev' into feat-New-gate
ThomasB-Colibri 05dd652
fix: adjust measure test
ThomasB-Colibri d262b97
chore: remove warnings on adjust measure
ThomasB-Colibri e647a6b
Merge branch 'dev' into feat-New-gate
MathieuG-Colibri 25b35c4
fix: eval expr & merge issues
ThomasB-Colibri 0a22af5
chore: Files formated
github-actions[bot] 2d5f512
feat: test qasm for u gates
ThomasB-Colibri f03ec49
Merge branch 'feat-New-gate' of github.com:ColibrITD-SAS/mpqp into fe…
ThomasB-Colibri 1f0013f
chore: Files formated
github-actions[bot] 4c14fd0
fix: merge conflict & missings gates in IQM gate set
ThomasB-Colibri 91316de
Merge branch 'dev' into feat-New-gate
ThomasB-Colibri 64b751e
fix: path for cirq
ThomasB-Colibri 7951bb9
chore: documentation added to sphinx
ThomasB-Colibri 3bc18b6
fix: prx params order in qasm
ThomasB-Colibri 4bc0783
fix: qasm path & warning removed
ThomasB-Colibri 6676e97
fix: repr for single qubit rotation gates
ThomasB-Colibri f7fdc8d
fix: test of repr for rotation
ThomasB-Colibri 33cfbbc
Update mpqp/core/instruction/measurement/expectation_value.py
ThomasB-Colibri 720b5c3
fix: import order & redundant imports deleted
ThomasB-Colibri c8eedf0
fix: check in resolve gates
ThomasB-Colibri File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -15,6 +15,7 @@ __pycache__/ | |
| .mypy_cache/ | ||
| .pytest_cache/ | ||
| .ipynb_checkpoints/ | ||
| venv/ | ||
|
|
||
| # IDE | ||
| .vscode/ | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,15 +1,15 @@ | ||
| # pyright: reportUnusedImport=false | ||
| from .instruction import Instruction | ||
| from .barrier import Barrier | ||
| from .gates import * | ||
| from .breakpoint import Breakpoint | ||
| from .gates import * | ||
| from .instruction import Instruction | ||
| from .measurement import ( | ||
| Basis, | ||
| ComputationalBasis, | ||
| HadamardBasis, | ||
| VariableSizeBasis, | ||
| BasisMeasure, | ||
| ComputationalBasis, | ||
| ExpectationMeasure, | ||
| Observable, | ||
| HadamardBasis, | ||
| Measure, | ||
| Observable, | ||
| VariableSizeBasis, | ||
| ) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel it's a mistake to remove this, because if we replaced by resolve_instruction in the workflow, when a gate is not a Composed gate or in the set of gates supported, nothing is raised and the gate just pass like this.
So by removing this we don't have anymore the security. Simple example is a circuit containing the gate T, trying to run it on IBMDevice.AER_SIMULATOR_STABILIZER would have returned a ValueError before explaining what is the problem. Instead of that, the process continues until it hits the Qiskit runner, which removes the security we had on mpqp side (and I still believe it's better than letting it go until the provider, can cost QPU time in remote case).
I believe this should be done also for all devices, and maybe this check can be added after we call resolve_gate, would be an option to fix that ? I let you judge
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done, I put a check after the call of resolve_instructions c8eedf0