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

Cleaning up commit history #4

Merged
merged 3 commits into from
Jun 14, 2023
Merged

Conversation

robotAstray
Copy link
Owner

Summary

Details and comments

urihan and others added 3 commits June 5, 2023 12:56
* Probability to Sample result type

* passes tox lint

* updates toxlint

* fixes observable z error but not passses elint

* unit test added

* final commit with tests
* fix: Fallback to JAQCD for device properties

* fix: reformat
* replacing AWSBraketJob with AmazonBraketTask

* run the notebooks containing AmazonBraketTask

* deprecation warning added for AWSBraketJob class

* formatting fixed

* re-add <JOB_ARN>

* fix breaking changes on /provider/__init__.py and __init__.py

* rename job_id to task_id argument for AmazonBraketTask()

* rename braket_jobs_states to braket_tasks_states

* add task_id method to access the task_id similar to the one we have in the JobV1 class

* adding tests for new class 'AmazonBraketTask' and old class 'AWSBraketJob'

* formatting test_braket_job.py

* correcting definition of AWSBraketJob

* rewriting line 184 in braket_job.py

* adding class docstring to AWSBraketJob

* standard import 'from warnings import warn'  placed before 'from braket.aws import AwsQuantumTask' (wrong-import-order)

* docstring fix in test_braket_job.py

* assert the job id in test_AWS_job() function in test_braket_job.py

* running how to notebook #0

* running how to notebook #1

* running how to notebook #2

* running how to notebook #3

* running how to notebook number 5

* running tutorial  notebook number 3

* tutorial number 0
@robotAstray robotAstray merged commit 43f821f into robotAstray:main Jun 14, 2023
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.

3 participants