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

Support the new DefaultQubit class in pennylane >=0.33.0 #1874

Closed
wants to merge 7 commits into from

Conversation

Andrew-S-Rosen
Copy link
Contributor

@Andrew-S-Rosen Andrew-S-Rosen commented Dec 6, 2023

  • I have added the tests to cover my changes.
  • I have updated the documentation and CHANGELOG accordingly.
  • I have read the CONTRIBUTING document.

Closes #1871.

Essentially, in pennylane<0.33.0, the DefaultQubit class has an operations and observables set of class attributes that need to be provided as shown here. This behavior was modified in pennylane==0.33.0 as shown here.

I have updated the QEDevice class in Covalent accordingly and modified the version of pennylane to be pennylane>=0.33.0 instead of pennylane>=0.31.1,<0.33.0.

@Andrew-S-Rosen Andrew-S-Rosen requested a review from a team as a code owner December 6, 2023 05:17
@Andrew-S-Rosen Andrew-S-Rosen requested a review from a team as a code owner December 6, 2023 05:18
Copy link

codecov bot commented Dec 6, 2023

Codecov Report

Merging #1874 (8f0f119) into develop (04efb9d) will increase coverage by 0.00%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #1874   +/-   ##
========================================
  Coverage    84.44%   84.44%           
========================================
  Files          294      294           
  Lines        14396    14393    -3     
  Branches       195      195           
========================================
- Hits         12156    12154    -2     
+ Misses        2106     2105    -1     
  Partials       134      134           
Flag Coverage Δ
Dispatcher 92.27% <ø> (ø)
Functional_Tests 51.27% <ø> (-0.01%) ⬇️
SDK 79.33% <ø> (-0.01%) ⬇️
UI_Backend 85.07% <ø> (ø)
UI_Frontend 73.35% <ø> (ø)

@Andrew-S-Rosen Andrew-S-Rosen changed the title [Not for merge] Find out where QEDevice.operations is being used Support the new DefaultQubit class in pennylane >=0.33.0 Dec 6, 2023
@kessler-frost
Copy link
Member

As commented here: #1871 (comment), this is currently blocked by #1877 .

@Andrew-S-Rosen Andrew-S-Rosen marked this pull request as draft December 7, 2023 05:43
@Andrew-S-Rosen
Copy link
Contributor Author

Closing because others will need to take over this one. 😉

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.

Support for pennylane >=0.33.0 (and thereby newer versions of numpy)
2 participants