amazon-braket / Braket.jl

Experimental Julia implementation of the Amazon Braket SDK
Apache License 2.0
97 stars 6 forks source link

fix: properly support IonQ native gates and add various tests #46

Closed kshyatt-aws closed 1 year ago

kshyatt-aws commented 1 year ago

Issue #, if available:

N/A

Description of changes:

Add support in IR module for the IonQ gates so they can be properly applied. Also add a variety of other small tests. Parameters are now written in sorted order to OpenQASM headers.

Testing done:

Unit tests passed locally.

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

General

Tests

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

codecov-commenter commented 1 year ago

Codecov Report

Base: 92.84% // Head: 84.33% // Decreases project coverage by -8.51% :warning:

Coverage data is based on head (8213e39) compared to base (7800860). Patch coverage: 50.00% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #46 +/- ## ========================================== - Coverage 92.84% 84.33% -8.51% ========================================== Files 33 33 Lines 3818 3825 +7 ========================================== - Hits 3545 3226 -319 - Misses 273 599 +326 ``` | [Impacted Files](https://codecov.io/gh/awslabs/Braket.jl/pull/46?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs) | Coverage Δ | | |---|---|---| | [src/gates.jl](https://codecov.io/gh/awslabs/Braket.jl/pull/46?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs#diff-c3JjL2dhdGVzLmps) | `86.04% <0.00%> (+13.10%)` | :arrow_up: | | [src/raw\_schema.jl](https://codecov.io/gh/awslabs/Braket.jl/pull/46?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs#diff-c3JjL3Jhd19zY2hlbWEuamw=) | `95.06% <50.00%> (-0.70%)` | :arrow_down: | | [src/circuit.jl](https://codecov.io/gh/awslabs/Braket.jl/pull/46?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs#diff-c3JjL2NpcmN1aXQuamw=) | `98.45% <100.00%> (+1.54%)` | :arrow_up: | | [PyBraket/src/pynoises.jl](https://codecov.io/gh/awslabs/Braket.jl/pull/46?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs#diff-UHlCcmFrZXQvc3JjL3B5bm9pc2VzLmps) | `0.00% <0.00%> (-100.00%)` | :arrow_down: | | [PyBraket/src/pygates.jl](https://codecov.io/gh/awslabs/Braket.jl/pull/46?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs#diff-UHlCcmFrZXQvc3JjL3B5Z2F0ZXMuamw=) | `10.29% <0.00%> (-86.77%)` | :arrow_down: | | [PyBraket/src/pycircuit.jl](https://codecov.io/gh/awslabs/Braket.jl/pull/46?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs#diff-UHlCcmFrZXQvc3JjL3B5Y2lyY3VpdC5qbA==) | `32.35% <0.00%> (-44.12%)` | :arrow_down: | | [src/operators.jl](https://codecov.io/gh/awslabs/Braket.jl/pull/46?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs#diff-c3JjL29wZXJhdG9ycy5qbA==) | `50.00% <0.00%> (-25.00%)` | :arrow_down: | | [PyBraket/src/local\_simulator.jl](https://codecov.io/gh/awslabs/Braket.jl/pull/46?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs#diff-UHlCcmFrZXQvc3JjL2xvY2FsX3NpbXVsYXRvci5qbA==) | `64.86% <0.00%> (-21.63%)` | :arrow_down: | | [PyBraket/src/PyBraket.jl](https://codecov.io/gh/awslabs/Braket.jl/pull/46?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs#diff-UHlCcmFrZXQvc3JjL1B5QnJha2V0Lmps) | `82.05% <0.00%> (-5.13%)` | :arrow_down: | | [PyBraket/src/pyschema.jl](https://codecov.io/gh/awslabs/Braket.jl/pull/46?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs#diff-UHlCcmFrZXQvc3JjL3B5c2NoZW1hLmps) | `96.11% <0.00%> (-2.27%)` | :arrow_down: | | ... and [10 more](https://codecov.io/gh/awslabs/Braket.jl/pull/46?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs) | | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.