Skip to content
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

feat: upgrade PL to allow Python 3.11 #148

Merged
merged 4 commits into from
Apr 24, 2023
Merged

feat: upgrade PL to allow Python 3.11 #148

merged 4 commits into from
Apr 24, 2023

Conversation

AbeCoull
Copy link
Contributor

Issue #, if available:

Description of changes:
Upgrade PL to allow Python 3.11.

Testing done:

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

  • I have added tests that prove my fix is effective or that my feature works (if appropriate)
  • I have checked that my tests are not configured for a specific region or account (if appropriate)

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

@AbeCoull AbeCoull requested a review from a team as a code owner March 29, 2023 21:56
@codecov
Copy link

codecov bot commented Mar 29, 2023

Codecov Report

Merging #148 (0cf8961) into main (cd461ab) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #148   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            5         5           
  Lines          657       657           
  Branches        93       164   +71     
=========================================
  Hits           657       657           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@AbeCoull AbeCoull requested a review from krneta April 20, 2023 17:34
@AbeCoull AbeCoull changed the title DO NOT MERGE: upgrade PL to allow Python 3.11 feat: upgrade PL to allow Python 3.11 Apr 21, 2023
@AbeCoull AbeCoull merged commit 4abce67 into main Apr 24, 2023
@AbeCoull AbeCoull deleted the py311 branch April 24, 2023 17:31
speller26 added a commit that referenced this pull request Apr 27, 2023
* feat: add Python 3.11 support (#148)

* feat: add Python 3.11 support

* prepare release v1.14.0

* update development version to v1.14.1.dev0

* test: parallelize test execution for pytest (#152)

* prepare release v1.14.1

* update development version to v1.14.2.dev0

---------

Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: ci <ci>
mudit2812 pushed a commit to lillian542/amazon-braket-pennylane-plugin-python that referenced this pull request May 8, 2023
mudit2812 added a commit to lillian542/amazon-braket-pennylane-plugin-python that referenced this pull request Jul 7, 2023
* Added `ParametrizedEvolution` support to OQC Lucy

* feat: add Python 3.11 support (amazon-braket#148)

* feat: add Python 3.11 support

* prepare release v1.14.0

* update development version to v1.14.1.dev0

* test: parallelize test execution for pytest (amazon-braket#152)

* prepare release v1.14.1

* update development version to v1.14.2.dev0

* feat: Add AHS devices (amazon-braket#158)

Also made plugin compatible with new return types specification (amazon-braket#153)

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Postprocess jacobian result shape (amazon-braket#159)

* fix: Tempfix for failing tests (amazon-braket#161)

* prepare release v1.15.0

* update development version to v1.15.1.dev0

* doc: Correct README format for PyPI (amazon-braket#162)

* prepare release v1.15.0.post0

* update development version to v1.15.1.dev0

* tests work weeeeee

* Fixed shadow expval error

---------

Co-authored-by: lillian542 <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>
mudit2812 added a commit to lillian542/amazon-braket-pennylane-plugin-python that referenced this pull request Jul 7, 2023
* Added `ParametrizedEvolution` support to OQC Lucy

* feat: add Python 3.11 support (amazon-braket#148)

* feat: add Python 3.11 support

* prepare release v1.14.0

* update development version to v1.14.1.dev0

* test: parallelize test execution for pytest (amazon-braket#152)

* prepare release v1.14.1

* update development version to v1.14.2.dev0

* feat: Add AHS devices (amazon-braket#158)

Also made plugin compatible with new return types specification (amazon-braket#153)

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Postprocess jacobian result shape (amazon-braket#159)

* fix: Tempfix for failing tests (amazon-braket#161)

* prepare release v1.15.0

* update development version to v1.15.1.dev0

* doc: Correct README format for PyPI (amazon-braket#162)

* prepare release v1.15.0.post0

* update development version to v1.15.1.dev0

* tests work weeeeee

* Fixed shadow expval error

---------

Co-authored-by: lillian542 <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>
mudit2812 added a commit to lillian542/amazon-braket-pennylane-plugin-python that referenced this pull request Jul 14, 2023
…on` (#13)

* Added `ParametrizedEvolution` support to OQC Lucy

* feat: add Python 3.11 support (amazon-braket#148)

* feat: add Python 3.11 support

* prepare release v1.14.0

* update development version to v1.14.1.dev0

* test: parallelize test execution for pytest (amazon-braket#152)

* prepare release v1.14.1

* update development version to v1.14.2.dev0

* feat: Add AHS devices (amazon-braket#158)

Also made plugin compatible with new return types specification (amazon-braket#153)

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Postprocess jacobian result shape (amazon-braket#159)

* fix: Tempfix for failing tests (amazon-braket#161)

* prepare release v1.15.0

* update development version to v1.15.1.dev0

* doc: Correct README format for PyPI (amazon-braket#162)

* prepare release v1.15.0.post0

* update development version to v1.15.1.dev0

* tests work weeeeee

* Added PE to PulseSequence translation

* Updated to support multiple pulses

* Added comments for readability

* Update src/braket/pennylane_plugin/translation.py

Co-authored-by: Korbinian Kottmann <[email protected]>

* Refactoring

* Added test for pulse gate translation

* Updated translation to stop fixing amplitude

* Reformatting

* Apply suggestions from code review

Co-authored-by: lillian542 <[email protected]>

* Added back translation for AAMS

* fix: constrain tensorflow version (amazon-braket#185)

* prepare release v1.17.1

* update development version to v1.17.2.dev0

* Add `ParametrizedEvolution` to OQC Lucy's supported operations (#10)

* Added `ParametrizedEvolution` support to OQC Lucy

* feat: add Python 3.11 support (amazon-braket#148)

* feat: add Python 3.11 support

* prepare release v1.14.0

* update development version to v1.14.1.dev0

* test: parallelize test execution for pytest (amazon-braket#152)

* prepare release v1.14.1

* update development version to v1.14.2.dev0

* feat: Add AHS devices (amazon-braket#158)

Also made plugin compatible with new return types specification (amazon-braket#153)

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Postprocess jacobian result shape (amazon-braket#159)

* fix: Tempfix for failing tests (amazon-braket#161)

* prepare release v1.15.0

* update development version to v1.15.1.dev0

* doc: Correct README format for PyPI (amazon-braket#162)

* prepare release v1.15.0.post0

* update development version to v1.15.1.dev0

* tests work weeeeee

* Fixed shadow expval error

---------

Co-authored-by: lillian542 <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Added unit tests

* Linting

* Updated return value of wires

* Updated variable name

* Update pennylane version constraint (amazon-braket#188)

* Update src/braket/pennylane_plugin/translation.py

Co-authored-by: lillian542 <[email protected]>

* prepare release v1.17.2

* update development version to v1.17.3.dev0

* Updated tests

* fix: set do_queue default to None (amazon-braket#175)

Co-authored-by: Cody Wang <[email protected]>

* prepare release v1.17.3

* update development version to v1.17.4.dev0

* Addressed code review

---------

Co-authored-by: lillian542 <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>
Co-authored-by: Stephen Face <[email protected]>
Co-authored-by: Kshitij Chhabra <[email protected]>
mudit2812 added a commit to lillian542/amazon-braket-pennylane-plugin-python that referenced this pull request Jul 25, 2023
* Updated translation to stop fixing amplitude

* Reformatting

* Apply suggestions from code review

Co-authored-by: lillian542 <[email protected]>

* Added back translation for AAMS

* fix: constrain tensorflow version (amazon-braket#185)

* prepare release v1.17.1

* update development version to v1.17.2.dev0

* Add `ParametrizedEvolution` to OQC Lucy's supported operations (#10)

* Added `ParametrizedEvolution` support to OQC Lucy

* feat: add Python 3.11 support (amazon-braket#148)

* feat: add Python 3.11 support

* prepare release v1.14.0

* update development version to v1.14.1.dev0

* test: parallelize test execution for pytest (amazon-braket#152)

* prepare release v1.14.1

* update development version to v1.14.2.dev0

* feat: Add AHS devices (amazon-braket#158)

Also made plugin compatible with new return types specification (amazon-braket#153)

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Postprocess jacobian result shape (amazon-braket#159)

* fix: Tempfix for failing tests (amazon-braket#161)

* prepare release v1.15.0

* update development version to v1.15.1.dev0

* doc: Correct README format for PyPI (amazon-braket#162)

* prepare release v1.15.0.post0

* update development version to v1.15.1.dev0

* tests work weeeeee

* Fixed shadow expval error

---------

Co-authored-by: lillian542 <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Added unit tests

* Linting

* Updated return value of wires

* Updated variable name

* Update src/braket/pennylane_plugin/translation.py

Co-authored-by: lillian542 <[email protected]>

* Updated tests

* pulse validation

* clean up after merge conflict

* warn if pulse settings don't match

* update validation functions

* add tests that mostly work

* working freq range test

* tox formatting

* allow qml.pulse.constant as constant parameters

* Apply suggestions from code review

* Added callable related changes to translation

* Fix compatibility with the new Projector class (amazon-braket#189)

* Fix attempt

* support tests

* Test error when not in Z basis

* fix formatting

* prepare release v1.17.4

* update development version to v1.17.5.dev0

* infra: Update CODEOWNERS (amazon-braket#184)

Co-authored-by: Abe Coull <[email protected]>

* feat: native mode (amazon-braket#187)

You can now specify `verbatim` when creating a device to run tasks in verbatim mode. This enables using native gates on QPUs.

* prepare release v1.18.0

* update development version to v1.18.1.dev0

---------

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Stephen Face <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>
Co-authored-by: BorjaRequena <[email protected]>
Co-authored-by: Aaron Berdy <[email protected]>
speller26 added a commit that referenced this pull request Aug 17, 2023
* Added `ParametrizedEvolution` support to OQC Lucy

* feat: add Python 3.11 support (#148)

* feat: add Python 3.11 support

* prepare release v1.14.0

* update development version to v1.14.1.dev0

* test: parallelize test execution for pytest (#152)

* prepare release v1.14.1

* update development version to v1.14.2.dev0

* feat: Add AHS devices (#158)

Also made plugin compatible with new return types specification (#153)

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Postprocess jacobian result shape (#159)

* fix: Tempfix for failing tests (#161)

* prepare release v1.15.0

* update development version to v1.15.1.dev0

* doc: Correct README format for PyPI (#162)

* prepare release v1.15.0.post0

* update development version to v1.15.1.dev0

* tests work weeeeee

* Added PE to PulseSequence translation

* Updated to support multiple pulses

* Added comments for readability

* Update src/braket/pennylane_plugin/translation.py

Co-authored-by: Korbinian Kottmann <[email protected]>

* Refactoring

* add anharmonicities

* Remove coupling from settings

* add tests

* tox formatting

* fix code formatting issues

* Added test for pulse gate translation

* apply suggestions from code review

* fix failing tests

* docstrings

* Updated translation to stop fixing amplitude

* Reformatting

* change name from settings to pulse_settings

* Apply suggestions from code review

Co-authored-by: lillian542 <[email protected]>

* Added back translation for AAMS

* Add `ParametrizedEvolution` to OQC Lucy's supported operations (#10)

* Added `ParametrizedEvolution` support to OQC Lucy

* feat: add Python 3.11 support (#148)

* feat: add Python 3.11 support

* prepare release v1.14.0

* update development version to v1.14.1.dev0

* test: parallelize test execution for pytest (#152)

* prepare release v1.14.1

* update development version to v1.14.2.dev0

* feat: Add AHS devices (#158)

Also made plugin compatible with new return types specification (#153)

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Postprocess jacobian result shape (#159)

* fix: Tempfix for failing tests (#161)

* prepare release v1.15.0

* update development version to v1.15.1.dev0

* doc: Correct README format for PyPI (#162)

* prepare release v1.15.0.post0

* update development version to v1.15.1.dev0

* tests work weeeeee

* Fixed shadow expval error

---------

Co-authored-by: lillian542 <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* fix: constrain tensorflow version (#185)

* prepare release v1.17.1

* update development version to v1.17.2.dev0

* Add `ParametrizedEvolution` to OQC Lucy's supported operations (#10)

* Added `ParametrizedEvolution` support to OQC Lucy

* feat: add Python 3.11 support (#148)

* feat: add Python 3.11 support

* prepare release v1.14.0

* update development version to v1.14.1.dev0

* test: parallelize test execution for pytest (#152)

* prepare release v1.14.1

* update development version to v1.14.2.dev0

* feat: Add AHS devices (#158)

Also made plugin compatible with new return types specification (#153)

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Postprocess jacobian result shape (#159)

* fix: Tempfix for failing tests (#161)

* prepare release v1.15.0

* update development version to v1.15.1.dev0

* doc: Correct README format for PyPI (#162)

* prepare release v1.15.0.post0

* update development version to v1.15.1.dev0

* tests work weeeeee

* Fixed shadow expval error

---------

Co-authored-by: lillian542 <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Added unit tests

* Linting

* Updated return value of wires

* Updated variable name

* Update src/braket/pennylane_plugin/translation.py

Co-authored-by: lillian542 <[email protected]>

* Updated tests

* pulse validation

* Add translation function to get `PulseGate` from `ParametrizedEvolution` (#13)

* Added `ParametrizedEvolution` support to OQC Lucy

* feat: add Python 3.11 support (#148)

* feat: add Python 3.11 support

* prepare release v1.14.0

* update development version to v1.14.1.dev0

* test: parallelize test execution for pytest (#152)

* prepare release v1.14.1

* update development version to v1.14.2.dev0

* feat: Add AHS devices (#158)

Also made plugin compatible with new return types specification (#153)

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Postprocess jacobian result shape (#159)

* fix: Tempfix for failing tests (#161)

* prepare release v1.15.0

* update development version to v1.15.1.dev0

* doc: Correct README format for PyPI (#162)

* prepare release v1.15.0.post0

* update development version to v1.15.1.dev0

* tests work weeeeee

* Added PE to PulseSequence translation

* Updated to support multiple pulses

* Added comments for readability

* Update src/braket/pennylane_plugin/translation.py

Co-authored-by: Korbinian Kottmann <[email protected]>

* Refactoring

* Added test for pulse gate translation

* Updated translation to stop fixing amplitude

* Reformatting

* Apply suggestions from code review

Co-authored-by: lillian542 <[email protected]>

* Added back translation for AAMS

* fix: constrain tensorflow version (#185)

* prepare release v1.17.1

* update development version to v1.17.2.dev0

* Add `ParametrizedEvolution` to OQC Lucy's supported operations (#10)

* Added `ParametrizedEvolution` support to OQC Lucy

* feat: add Python 3.11 support (#148)

* feat: add Python 3.11 support

* prepare release v1.14.0

* update development version to v1.14.1.dev0

* test: parallelize test execution for pytest (#152)

* prepare release v1.14.1

* update development version to v1.14.2.dev0

* feat: Add AHS devices (#158)

Also made plugin compatible with new return types specification (#153)

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Postprocess jacobian result shape (#159)

* fix: Tempfix for failing tests (#161)

* prepare release v1.15.0

* update development version to v1.15.1.dev0

* doc: Correct README format for PyPI (#162)

* prepare release v1.15.0.post0

* update development version to v1.15.1.dev0

* tests work weeeeee

* Fixed shadow expval error

---------

Co-authored-by: lillian542 <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Added unit tests

* Linting

* Updated return value of wires

* Updated variable name

* Update pennylane version constraint (#188)

* Update src/braket/pennylane_plugin/translation.py

Co-authored-by: lillian542 <[email protected]>

* prepare release v1.17.2

* update development version to v1.17.3.dev0

* Updated tests

* fix: set do_queue default to None (#175)

Co-authored-by: Cody Wang <[email protected]>

* prepare release v1.17.3

* update development version to v1.17.4.dev0

* Addressed code review

---------

Co-authored-by: lillian542 <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>
Co-authored-by: Stephen Face <[email protected]>
Co-authored-by: Kshitij Chhabra <[email protected]>

* remove unwanted changes to translation.py

* revert unwanted changes to shadow_expval tests

* fix typo from last commit

* remove unwanted changes to test_translation

* linting

* more lint

* raise clear NotImplementedError if pulse properties are called on other devices

* update tests

* Fix formatting

* clean up after merge conflict

* warn if pulse settings don't match

* update validation functions

* add tests that mostly work

* working freq range test

* tox formatting

* allow qml.pulse.constant as constant parameters

* Apply suggestions from code review

* Added callable related changes to translation

* Fix compatibility with the new Projector class (#189)

* Fix attempt

* support tests

* Test error when not in Z basis

* fix formatting

* prepare release v1.17.4

* update development version to v1.17.5.dev0

* infra: Update CODEOWNERS (#184)

Co-authored-by: Abe Coull <[email protected]>

* feat: native mode (#187)

You can now specify `verbatim` when creating a device to run tasks in verbatim mode. This enables using native gates on QPUs.

* prepare release v1.18.0

* update development version to v1.18.1.dev0

* Adding docs

* Updated docs

* Updated white spacing

* Updated docs

* Pulse validation (#14)

* Updated translation to stop fixing amplitude

* Reformatting

* Apply suggestions from code review

Co-authored-by: lillian542 <[email protected]>

* Added back translation for AAMS

* fix: constrain tensorflow version (#185)

* prepare release v1.17.1

* update development version to v1.17.2.dev0

* Add `ParametrizedEvolution` to OQC Lucy's supported operations (#10)

* Added `ParametrizedEvolution` support to OQC Lucy

* feat: add Python 3.11 support (#148)

* feat: add Python 3.11 support

* prepare release v1.14.0

* update development version to v1.14.1.dev0

* test: parallelize test execution for pytest (#152)

* prepare release v1.14.1

* update development version to v1.14.2.dev0

* feat: Add AHS devices (#158)

Also made plugin compatible with new return types specification (#153)

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Postprocess jacobian result shape (#159)

* fix: Tempfix for failing tests (#161)

* prepare release v1.15.0

* update development version to v1.15.1.dev0

* doc: Correct README format for PyPI (#162)

* prepare release v1.15.0.post0

* update development version to v1.15.1.dev0

* tests work weeeeee

* Fixed shadow expval error

---------

Co-authored-by: lillian542 <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Added unit tests

* Linting

* Updated return value of wires

* Updated variable name

* Update src/braket/pennylane_plugin/translation.py

Co-authored-by: lillian542 <[email protected]>

* Updated tests

* pulse validation

* clean up after merge conflict

* warn if pulse settings don't match

* update validation functions

* add tests that mostly work

* working freq range test

* tox formatting

* allow qml.pulse.constant as constant parameters

* Apply suggestions from code review

* Added callable related changes to translation

* Fix compatibility with the new Projector class (#189)

* Fix attempt

* support tests

* Test error when not in Z basis

* fix formatting

* prepare release v1.17.4

* update development version to v1.17.5.dev0

* infra: Update CODEOWNERS (#184)

Co-authored-by: Abe Coull <[email protected]>

* feat: native mode (#187)

You can now specify `verbatim` when creating a device to run tasks in verbatim mode. This enables using native gates on QPUs.

* prepare release v1.18.0

* update development version to v1.18.1.dev0

---------

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Stephen Face <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>
Co-authored-by: BorjaRequena <[email protected]>
Co-authored-by: Aaron Berdy <[email protected]>

* remove custom wire logic

* use AwsDevice instead of BraketAwsQubitDevice

* remove stray print statement

* use ConstantWaveform for pulse.constant

* chronological wire ordering

* update docstring

* further clarify docstring

* clarify comment in validate_pulse_parameters

* update _is_single_qubit_12_frame error message

* make device required for ParametrizedEvolution dispatch

* update documentation

* black

* update check_validity

* received

* update units documentation

* fix formatting

* add integration test

* black formatting

* handle device offline in integ tests

* remove unused import in test file

---------

Co-authored-by: lillian542 <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Stephen Face <[email protected]>
Co-authored-by: Kshitij Chhabra <[email protected]>
Co-authored-by: BorjaRequena <[email protected]>
Co-authored-by: Aaron Berdy <[email protected]>
kshitijc added a commit that referenced this pull request Oct 24, 2023
* Added `ParametrizedEvolution` support to OQC Lucy

* feat: add Python 3.11 support (#148)

* feat: add Python 3.11 support

* prepare release v1.14.0

* update development version to v1.14.1.dev0

* test: parallelize test execution for pytest (#152)

* prepare release v1.14.1

* update development version to v1.14.2.dev0

* feat: Add AHS devices (#158)

Also made plugin compatible with new return types specification (#153)

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Postprocess jacobian result shape (#159)

* fix: Tempfix for failing tests (#161)

* prepare release v1.15.0

* update development version to v1.15.1.dev0

* doc: Correct README format for PyPI (#162)

* prepare release v1.15.0.post0

* update development version to v1.15.1.dev0

* tests work weeeeee

* Added PE to PulseSequence translation

* Updated to support multiple pulses

* Added comments for readability

* Update src/braket/pennylane_plugin/translation.py

Co-authored-by: Korbinian Kottmann <[email protected]>

* Refactoring

* add anharmonicities

* Remove coupling from settings

* add tests

* tox formatting

* fix code formatting issues

* Added test for pulse gate translation

* apply suggestions from code review

* fix failing tests

* docstrings

* Updated translation to stop fixing amplitude

* Reformatting

* change name from settings to pulse_settings

* Apply suggestions from code review

Co-authored-by: lillian542 <[email protected]>

* Added back translation for AAMS

* Add `ParametrizedEvolution` to OQC Lucy's supported operations (#10)

* Added `ParametrizedEvolution` support to OQC Lucy

* feat: add Python 3.11 support (#148)

* feat: add Python 3.11 support

* prepare release v1.14.0

* update development version to v1.14.1.dev0

* test: parallelize test execution for pytest (#152)

* prepare release v1.14.1

* update development version to v1.14.2.dev0

* feat: Add AHS devices (#158)

Also made plugin compatible with new return types specification (#153)

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Postprocess jacobian result shape (#159)

* fix: Tempfix for failing tests (#161)

* prepare release v1.15.0

* update development version to v1.15.1.dev0

* doc: Correct README format for PyPI (#162)

* prepare release v1.15.0.post0

* update development version to v1.15.1.dev0

* tests work weeeeee

* Fixed shadow expval error

---------

Co-authored-by: lillian542 <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* fix: constrain tensorflow version (#185)

* prepare release v1.17.1

* update development version to v1.17.2.dev0

* Add `ParametrizedEvolution` to OQC Lucy's supported operations (#10)

* Added `ParametrizedEvolution` support to OQC Lucy

* feat: add Python 3.11 support (#148)

* feat: add Python 3.11 support

* prepare release v1.14.0

* update development version to v1.14.1.dev0

* test: parallelize test execution for pytest (#152)

* prepare release v1.14.1

* update development version to v1.14.2.dev0

* feat: Add AHS devices (#158)

Also made plugin compatible with new return types specification (#153)

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Postprocess jacobian result shape (#159)

* fix: Tempfix for failing tests (#161)

* prepare release v1.15.0

* update development version to v1.15.1.dev0

* doc: Correct README format for PyPI (#162)

* prepare release v1.15.0.post0

* update development version to v1.15.1.dev0

* tests work weeeeee

* Fixed shadow expval error

---------

Co-authored-by: lillian542 <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Added unit tests

* Linting

* Updated return value of wires

* Updated variable name

* Update src/braket/pennylane_plugin/translation.py

Co-authored-by: lillian542 <[email protected]>

* Updated tests

* pulse validation

* Add translation function to get `PulseGate` from `ParametrizedEvolution` (#13)

* Added `ParametrizedEvolution` support to OQC Lucy

* feat: add Python 3.11 support (#148)

* feat: add Python 3.11 support

* prepare release v1.14.0

* update development version to v1.14.1.dev0

* test: parallelize test execution for pytest (#152)

* prepare release v1.14.1

* update development version to v1.14.2.dev0

* feat: Add AHS devices (#158)

Also made plugin compatible with new return types specification (#153)

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Postprocess jacobian result shape (#159)

* fix: Tempfix for failing tests (#161)

* prepare release v1.15.0

* update development version to v1.15.1.dev0

* doc: Correct README format for PyPI (#162)

* prepare release v1.15.0.post0

* update development version to v1.15.1.dev0

* tests work weeeeee

* Added PE to PulseSequence translation

* Updated to support multiple pulses

* Added comments for readability

* Update src/braket/pennylane_plugin/translation.py

Co-authored-by: Korbinian Kottmann <[email protected]>

* Refactoring

* Added test for pulse gate translation

* Updated translation to stop fixing amplitude

* Reformatting

* Apply suggestions from code review

Co-authored-by: lillian542 <[email protected]>

* Added back translation for AAMS

* fix: constrain tensorflow version (#185)

* prepare release v1.17.1

* update development version to v1.17.2.dev0

* Add `ParametrizedEvolution` to OQC Lucy's supported operations (#10)

* Added `ParametrizedEvolution` support to OQC Lucy

* feat: add Python 3.11 support (#148)

* feat: add Python 3.11 support

* prepare release v1.14.0

* update development version to v1.14.1.dev0

* test: parallelize test execution for pytest (#152)

* prepare release v1.14.1

* update development version to v1.14.2.dev0

* feat: Add AHS devices (#158)

Also made plugin compatible with new return types specification (#153)

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Postprocess jacobian result shape (#159)

* fix: Tempfix for failing tests (#161)

* prepare release v1.15.0

* update development version to v1.15.1.dev0

* doc: Correct README format for PyPI (#162)

* prepare release v1.15.0.post0

* update development version to v1.15.1.dev0

* tests work weeeeee

* Fixed shadow expval error

---------

Co-authored-by: lillian542 <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Added unit tests

* Linting

* Updated return value of wires

* Updated variable name

* Update pennylane version constraint (#188)

* Update src/braket/pennylane_plugin/translation.py

Co-authored-by: lillian542 <[email protected]>

* prepare release v1.17.2

* update development version to v1.17.3.dev0

* Updated tests

* fix: set do_queue default to None (#175)

Co-authored-by: Cody Wang <[email protected]>

* prepare release v1.17.3

* update development version to v1.17.4.dev0

* Addressed code review

---------

Co-authored-by: lillian542 <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>
Co-authored-by: Stephen Face <[email protected]>
Co-authored-by: Kshitij Chhabra <[email protected]>

* remove unwanted changes to translation.py

* revert unwanted changes to shadow_expval tests

* fix typo from last commit

* remove unwanted changes to test_translation

* linting

* more lint

* raise clear NotImplementedError if pulse properties are called on other devices

* update tests

* Fix formatting

* clean up after merge conflict

* warn if pulse settings don't match

* update validation functions

* add tests that mostly work

* working freq range test

* tox formatting

* allow qml.pulse.constant as constant parameters

* Apply suggestions from code review

* Added callable related changes to translation

* Fix compatibility with the new Projector class (#189)

* Fix attempt

* support tests

* Test error when not in Z basis

* fix formatting

* prepare release v1.17.4

* update development version to v1.17.5.dev0

* infra: Update CODEOWNERS (#184)

Co-authored-by: Abe Coull <[email protected]>

* feat: native mode (#187)

You can now specify `verbatim` when creating a device to run tasks in verbatim mode. This enables using native gates on QPUs.

* prepare release v1.18.0

* update development version to v1.18.1.dev0

* Adding docs

* Updated docs

* Updated white spacing

* Updated docs

* Pulse validation (#14)

* Updated translation to stop fixing amplitude

* Reformatting

* Apply suggestions from code review

Co-authored-by: lillian542 <[email protected]>

* Added back translation for AAMS

* fix: constrain tensorflow version (#185)

* prepare release v1.17.1

* update development version to v1.17.2.dev0

* Add `ParametrizedEvolution` to OQC Lucy's supported operations (#10)

* Added `ParametrizedEvolution` support to OQC Lucy

* feat: add Python 3.11 support (#148)

* feat: add Python 3.11 support

* prepare release v1.14.0

* update development version to v1.14.1.dev0

* test: parallelize test execution for pytest (#152)

* prepare release v1.14.1

* update development version to v1.14.2.dev0

* feat: Add AHS devices (#158)

Also made plugin compatible with new return types specification (#153)

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Postprocess jacobian result shape (#159)

* fix: Tempfix for failing tests (#161)

* prepare release v1.15.0

* update development version to v1.15.1.dev0

* doc: Correct README format for PyPI (#162)

* prepare release v1.15.0.post0

* update development version to v1.15.1.dev0

* tests work weeeeee

* Fixed shadow expval error

---------

Co-authored-by: lillian542 <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

* Added unit tests

* Linting

* Updated return value of wires

* Updated variable name

* Update src/braket/pennylane_plugin/translation.py

Co-authored-by: lillian542 <[email protected]>

* Updated tests

* pulse validation

* clean up after merge conflict

* warn if pulse settings don't match

* update validation functions

* add tests that mostly work

* working freq range test

* tox formatting

* allow qml.pulse.constant as constant parameters

* Apply suggestions from code review

* Added callable related changes to translation

* Fix compatibility with the new Projector class (#189)

* Fix attempt

* support tests

* Test error when not in Z basis

* fix formatting

* prepare release v1.17.4

* update development version to v1.17.5.dev0

* infra: Update CODEOWNERS (#184)

Co-authored-by: Abe Coull <[email protected]>

* feat: native mode (#187)

You can now specify `verbatim` when creating a device to run tasks in verbatim mode. This enables using native gates on QPUs.

* prepare release v1.18.0

* update development version to v1.18.1.dev0

---------

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Stephen Face <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>
Co-authored-by: BorjaRequena <[email protected]>
Co-authored-by: Aaron Berdy <[email protected]>

* remove custom wire logic

* use AwsDevice instead of BraketAwsQubitDevice

* remove stray print statement

* use ConstantWaveform for pulse.constant

* chronological wire ordering

* update docstring

* further clarify docstring

* clarify comment in validate_pulse_parameters

* update _is_single_qubit_12_frame error message

* make device required for ParametrizedEvolution dispatch

* update documentation

* black

* update check_validity

* received

* update units documentation

* fix formatting

* add integration test

* black formatting

* handle device offline in integ tests

* remove unused import in test file

* set frequency range to 3-8 GHz

* replace self.wires with wires

* update error and tests for frequency range

* use frame-specific dt value

* tox/formatting clean up

* move json dumps to seperate file pt1

* move json dumps to seperate file pt2

* remove warning when Ev operator contains interaction term

* flake8 and black formatting

* Update to use shift_phase

Co-authored-by: Jean-Christophe Jaskula <[email protected]>

* Update src/braket/pennylane_plugin/translation.py

* Update src/braket/pennylane_plugin/translation.py

* Update src/braket/pennylane_plugin/translation.py

* remove unused imports

---------

Co-authored-by: lillian542 <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: ci <ci>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Cody Wang <[email protected]>
Co-authored-by: Stephen Face <[email protected]>
Co-authored-by: Kshitij Chhabra <[email protected]>
Co-authored-by: BorjaRequena <[email protected]>
Co-authored-by: Aaron Berdy <[email protected]>
Co-authored-by: Christian Bruun Madsen <[email protected]>
Co-authored-by: Jean-Christophe Jaskula <[email protected]>
Co-authored-by: Qottmann <[email protected]>
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