feat: LHR region configuration #306

AbeCoull · 2022-02-27T21:47:01Z

Issue #, if available:

Description of changes:
feat: Oqc release
Testing done:
tox, tox integ tests

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

[ x ] I have read the CONTRIBUTING doc
[ x ] I used the commit message format described in CONTRIBUTING
[ x ] I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

[ x ] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[ x ] 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.

… range.

Co-authored-by: Aaron Berdy <[email protected]>

feature: add utility class for getting jobs metrics

Jobs Initial Commit

fix: better exception handling for aws device

* feature: Add support for executing OpenQasmProgram * test updates * Update branch for checks * integ tests * Result types calculation for openqasm programs (#164) * fix: Result types calculation for openqasm * Add tests * Update assert * Reformat Co-authored-by: Cody Wang <[email protected]> Co-authored-by: ℂ𝓞𝔇𝚈 <[email protected]>

* feature: add ECR gate * ECR as DoubleTarget * move ECR after CZ, remove branch from tox deps

* infra: Pin docutils<0.18 in doc requirements (#283) docutils 0.18 was released on October 26, 2021, and with it came some [breaking changes](readthedocs/readthedocs.org#8616) for sphinx, and in turn [readthedocs builds](https://readthedocs.org/projects/amazon-braket-sdk-python/builds/15168485/). To keep doc builds working, docutils will be constrained to versions below 0.18. * prepare release v1.9.5.post0 * update development version to v1.9.6.dev0 * feature: Add support for jobs (#287) Co-authored-by: Viraj Chaudhari <[email protected]> Co-authored-by: Milan Krneta <[email protected]> Co-authored-by: Aaron Berdy <[email protected]> Co-authored-by: Lin <[email protected]> Co-authored-by: Roald Bradley Severtson <[email protected]> Co-authored-by: Christian Madsen <[email protected]> * fix: Skip jobs integration tests (#288) * prepare release v1.10.0 * update development version to v1.10.1.dev0 * feature: Adding integration tests for DM1 (#286) * feature: Adding integration tests for DM1 * Moved many_layers to test_quantum_task * formatting changes only Co-authored-by: Cody Wang <[email protected]> * prepare release v1.11.0 * update development version to v1.11.1.dev0 * fix: remove extraneous reference from local job container setup (#292) * prepare release v1.11.1 * update development version to v1.11.2.dev0 * feature: optimize IAM role retrieval (#299) * fix: Enable jobs integration tests (#289) * feature: Added is_available property to AwsDevice (#290) Added is_available property to AwsDevice that parses the availability window and current device status to return a boolean flag if the device is currently available. * prepare release v1.12.0 * update development version to v1.12.1.dev0 * feature: added controlled-sqrt-not gate (#297) * feature: added controlled-sqrt-not gate This makes certain circuits, like CHSH, more straightforward. This commit works in line with the following branches (also committed, separately): https://github.com/unprovable/amazon-braket-schemas-python.git@ctrl-v-gate https://github.com/unprovable/amazon-braket-default-simulator-python.git@ctrl-v-gate * fix: ran tox linters * fix: reverted tox.ini Co-authored-by: Mark C <[email protected]> Co-authored-by: Mark C <[email protected]> Co-authored-by: Cody Wang <[email protected]> * prepare release v1.13.0 * update development version to v1.13.1.dev0 * feature: adding TwoQubitPauliChannel (#300) * prepare release v1.14.0 * update development version to v1.14.1.dev0 Co-authored-by: Cody Wang <[email protected]> Co-authored-by: ci <ci> Co-authored-by: Kshitij Chhabra <[email protected]> Co-authored-by: Milan Krneta <[email protected]> Co-authored-by: Aaron Berdy <[email protected]> Co-authored-by: Lin <[email protected]> Co-authored-by: Roald Bradley Severtson <[email protected]> Co-authored-by: Christian Madsen <[email protected]> Co-authored-by: Jacob Feldman <[email protected]> Co-authored-by: Mark Sweat <[email protected]> Co-authored-by: Mark C <[email protected]> Co-authored-by: Mark C <[email protected]> Co-authored-by: Mark C <[email protected]> Co-authored-by: mbeach-aws <[email protected]>

This reverts commit be2a2be.

This reverts commit c12c432.

* fix: Add LHR to the list of device regions * test updates * fix: Add integration tests * formatting changes

* infra: Pin docutils<0.18 in doc requirements (#283) docutils 0.18 was released on October 26, 2021, and with it came some [breaking changes](readthedocs/readthedocs.org#8616) for sphinx, and in turn [readthedocs builds](https://readthedocs.org/projects/amazon-braket-sdk-python/builds/15168485/). To keep doc builds working, docutils will be constrained to versions below 0.18. * prepare release v1.9.5.post0 * update development version to v1.9.6.dev0 * feature: Add support for jobs (#287) Co-authored-by: Viraj Chaudhari <[email protected]> Co-authored-by: Milan Krneta <[email protected]> Co-authored-by: Aaron Berdy <[email protected]> Co-authored-by: Lin <[email protected]> Co-authored-by: Roald Bradley Severtson <[email protected]> Co-authored-by: Christian Madsen <[email protected]> * fix: Skip jobs integration tests (#288) * prepare release v1.10.0 * update development version to v1.10.1.dev0 * feature: Adding integration tests for DM1 (#286) * feature: Adding integration tests for DM1 * Moved many_layers to test_quantum_task * formatting changes only Co-authored-by: Cody Wang <[email protected]> * prepare release v1.11.0 * update development version to v1.11.1.dev0 * fix: remove extraneous reference from local job container setup (#292) * prepare release v1.11.1 * update development version to v1.11.2.dev0 * feature: optimize IAM role retrieval (#299) * fix: Enable jobs integration tests (#289) * feature: Added is_available property to AwsDevice (#290) Added is_available property to AwsDevice that parses the availability window and current device status to return a boolean flag if the device is currently available. * prepare release v1.12.0 * update development version to v1.12.1.dev0 * feature: added controlled-sqrt-not gate (#297) * feature: added controlled-sqrt-not gate This makes certain circuits, like CHSH, more straightforward. This commit works in line with the following branches (also committed, separately): https://github.com/unprovable/amazon-braket-schemas-python.git@ctrl-v-gate https://github.com/unprovable/amazon-braket-default-simulator-python.git@ctrl-v-gate * fix: ran tox linters * fix: reverted tox.ini Co-authored-by: Mark C <[email protected]> Co-authored-by: Mark C <[email protected]> Co-authored-by: Cody Wang <[email protected]> * prepare release v1.13.0 * update development version to v1.13.1.dev0 * feature: adding TwoQubitPauliChannel (#300) * prepare release v1.14.0 * update development version to v1.14.1.dev0 * documentation: fix documentation on environment variable to match the code. (#302) * prepare release v1.14.0.post0 * update development version to v1.14.1.dev0 * feat: Update region switching for regional device arns (#169) (#303) * prepare release v1.15.0 * update development version to v1.15.1.dev0 Co-authored-by: Cody Wang <[email protected]> Co-authored-by: ci <ci> Co-authored-by: Viraj Chaudhari <[email protected]> Co-authored-by: Milan Krneta <[email protected]> Co-authored-by: Aaron Berdy <[email protected]> Co-authored-by: Lin <[email protected]> Co-authored-by: Roald Bradley Severtson <[email protected]> Co-authored-by: Christian Madsen <[email protected]> Co-authored-by: Jacob Feldman <[email protected]> Co-authored-by: Mark Sweat <[email protected]> Co-authored-by: Mark C <[email protected]> Co-authored-by: Mark C <[email protected]> Co-authored-by: Mark C <[email protected]> Co-authored-by: mbeach-aws <[email protected]> Co-authored-by: Yiheng Duan <[email protected]>

codecov · 2022-02-27T21:49:37Z

Codecov Report

Merging #306 (999057f) into main (a04f2e3) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #306   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           63        63           
  Lines         4335      4335           
  Branches       597       597           
=========================================
  Hits          4335      4335

Impacted Files	Coverage Δ
src/braket/aws/aws_device.py	`100.00% <100.00%> (ø)`

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a04f2e3...999057f. Read the comment docs.

* feat: queue depth for devices (dataclass version) * add unit-test for queue depth * modify docstrings, remove helper funcs * add more info to docstrings * minor test edit * docstrings * indent * update QueuePriority to QueueType

* feat: add queue position for tasks (#299) * feat: add queue position for tasks * update docstring for queue_position * update docstrings * update package_name * update to enums and dataclass * test: add integ test for task queue_position (#300) * refactor: dataclass and file naming for queue info (#301) * refactor: dataclass and file naming for queue info * apply suggestions * update: task queue position after refactor (#309) * update: task queue position after refactor * add queue_position type hint details, change order of info * feat: queue position for hybrid jobs (#302) * feat: queue position for hybrid jobs * handle message return * add docstring changes * update docstring, and return None * add context in dataclass * update docstrings * indent fix * test: add integ test for jobs queue position (#310) * feat: queue position for hybrid jobs * handle message return * add docstring changes * update docstring, and return None * add context in dataclass * test: add integ test for jobs queue position * remove comment * minor fix * remove unnecessary branching * fix docstring merge * remove dataclass redefinition after merge * feat: queue depth for devices (#306) * feat: queue depth for devices (dataclass version) * add unit-test for queue depth * modify docstrings, remove helper funcs * add more info to docstrings * minor test edit * docstrings * indent * update QueuePriority to QueueType * test: add integ test for queue_depth (#311) * refactor: job and quantum_task keywords for queue_depth (#312) * refactor: job and quantum_task keywords for queue_depth * Update src/braket/aws/queue_information.py Co-authored-by: Kshitij Chhabra <[email protected]> --------- Co-authored-by: Kshitij Chhabra <[email protected]> * deps: update boto3 version for queue visibility (#319) * sync: public-main changes into feature/queue_visibility (#320) * feat: add Aria2 enum (#653) Co-authored-by: Viraj Chaudhari <[email protected]> Co-authored-by: Cody Wang <[email protected]> * infra: bump actions/checkout from 3.6.0 to 4.0.0 (#696) Bumps [actions/checkout](https://github.com/actions/checkout) from 3.6.0 to 4.0.0. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@f43a0e5...3df4ab1) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * prepare release v1.55.0 * update development version to v1.55.1.dev0 * Revert "update: restricting parameter names to not collide with ones we use for OpenQASM generation. (#675)" (#701) This reverts commit b158736. * infra: update codeowner file to amazon-braket/braket-dev (#699) Co-authored-by: Abe Coull <[email protected]> * doc: Replace aws org with amazon-braket (#705) * prepare release v1.55.1 * update development version to v1.55.2.dev0 * doc: change the sphinx requirement to be greater than 7.0.0 (#704) Co-authored-by: Cody Wang <[email protected]> * doc: add code contributors to the readme (#703) Co-authored-by: Cody Wang <[email protected]> * doc: Remove trailing backquotes (#706) * infra: update the pre-commit hook with linters (#678) * infra: update the pre-commit hook with linters and secrets check Co-authored-by: Abe Coull <[email protected]> Co-authored-by: Cody Wang <[email protected]> * prepare release v1.55.1.post0 * update development version to v1.55.2.dev0 --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: ashlhans <[email protected]> Co-authored-by: Cody Wang <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: ci <ci> Co-authored-by: Milan <[email protected]> Co-authored-by: Angela Guo <[email protected]> Co-authored-by: Abe Coull <[email protected]> Co-authored-by: Abe Coull <[email protected]> * Revert "sync: public-main changes into feature/queue_visibility (#320)" This reverts commit be6460c. * update github script * minor fix --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Kshitij Chhabra <[email protected]> Co-authored-by: ashlhans <[email protected]> Co-authored-by: Cody Wang <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Milan <[email protected]> Co-authored-by: Angela Guo <[email protected]> Co-authored-by: Abe Coull <[email protected]> Co-authored-by: Abe Coull <[email protected]>

virajvchaudhari and others added 30 commits June 15, 2021 13:19

Jobs Initial Commit

d909ceb

Address Suggestions

9403ca0

Remove create job from aws_device

af783d0

Fix formatting

eb2957a

feature: add utility class for getting jobs metrics

7b0a9cd

feature: Code structure changes, docstrings fix

198d589

Remove implementation, duplicate content

870bbbd

Address suggestions

a228ca9

cfix default_bucket comment

ee9b682

Add copyright notice

c5b094f

incorporating feedback.

b18a202

Modifying the doc for returning the CW client, and updating copyright…

9b9c059

… range.

Forgot two copyright notices.

5f02948

Remove year from copyright content

bf4e211

Co-authored-by: Aaron Berdy <[email protected]>

Update PersistedJobData format

de51439

More fixes per PR comments.

cbae744

Adding a metrics fetcher that gets logs directly from CloudWatch logs.

345ae05

Updating braket jobs log group name.

a352320

Fixing comment in one of the test cases.

6e5b248

Fix defaults, Enums, comments

41c3acf

Merge pull request #2 from aws/jobmetrics

f427f08

feature: add utility class for getting jobs metrics

Fix comments, update jobdataformat enum

e0751c2

Merge branch 'main' into jobs

2dbc193

Handle imports

22cf0a7

Fix TypeError

ed419fd

Add TODO for metrics

c31b29d

feature: Jobs Skeleton

9c2d694

Jobs Initial Commit

feature: creating hybrid jobs from the Braket SDK

4d1d132

whitepace

7484311

some refactors mostly

06607f9

ajberdy and others added 21 commits November 19, 2021 11:09

Aws device fix (#161)

a90cf35

fix: better exception handling for aws device

resolve integ test todos (#156)

79521d2

fix: Remove CI updates for staging repo (#151)

23b4dbc

Sync with public repository (#162)

18ee1a1

fix: Update parameter to integration test

be2a2be

fix: formatting (#167)

6196262

feature: add ECR gate (#166)

e1f9663

* feature: add ECR gate * ECR as DoubleTarget * move ECR after CZ, remove branch from tox deps

feature: qubit limit for probability result type target (#170)

e0b8d6a

feat: Update region switching for regional device arns (#169)

ea6a654

fix: Use OqcDeviceParameters for Oqc devices

54bbb99

fix: Add configuration for LHR container images

d29f280

Revert "fix: Update parameter to integration test"

3d6e30d

This reverts commit be2a2be.

Revert "feature: Add support for executing OpenQasmPrograms (#116)"

62a62f3

This reverts commit c12c432.

Test fixture update

8e864b6

Added integration tests and LHR region to device configuration (#174)

4939b28

* fix: Add LHR to the list of device regions * test updates * fix: Add integration tests * formatting changes

fix: Return DiGraph for device.topology_graph (#175)

f3e1e46

fix: Remove CI updates for staging repo (#151)

98bc3b2

fix: OQC merge conflicts

999057f

AbeCoull requested a review from a team as a code owner February 27, 2022 21:47

kshitijc changed the title ~~feat: Oqc release~~ feat: LHR region configuration Feb 27, 2022

kshitijc approved these changes Feb 27, 2022

View reviewed changes

rchilaka-amzn approved these changes Feb 27, 2022

View reviewed changes

AbeCoull merged commit a89c2a5 into main Feb 27, 2022

AbeCoull deleted the oqc_release branch February 27, 2022 21:52

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: LHR region configuration #306

feat: LHR region configuration #306

AbeCoull commented Feb 27, 2022

codecov bot commented Feb 27, 2022 •

edited

Loading

feat: LHR region configuration #306

feat: LHR region configuration #306

Conversation

AbeCoull commented Feb 27, 2022

Merge Checklist

General

Tests

codecov bot commented Feb 27, 2022 • edited Loading

Codecov Report

codecov bot commented Feb 27, 2022 •

edited

Loading