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

[Andrew-S-Rosen]

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

[kessler-frost]

@Andrew-S-Rosen so when creating a new device in Pennylane there are some attributes that the custom device is required to have, essentially describing what are all the operations and observables that this particular device supports as not all devices can physically support all operations/observables. This is why here we are simply making our device support whatever the default device supports. This is why you couldn't find it in the codebase 😅 because it's something that is required by pennylane when defining a custom device.

[Andrew-S-Rosen]

@kessler-frost: Makes perfect sense to me! I was admittedly relying blind on the test suite for this since I don't know anything about pennylane. I'll go ahead and close this!

Edit: Oops, I thought this was a PR. The issue should still be kept open until it's resolved!

[kessler-frost]

I'd say even your PR should be open 😅 . I can comment on there once it's resolved. Until then you can keep it in draft mode instead I believe.

[Andrew-S-Rosen]

Looks like this is the offending line:

covalent/covalent/_workflow/qdevice.py

Line 45 in e45bd30

operations = DefaultQubit.operations

@kessler-frost: Is the QEDevice.operations attribute actually used anywhere? I couldn't spot it from a quick search in the codebase, but maybe I missed it.

[Andrew-S-Rosen]

@kessler-frost: Ah, that explains it! So then in order to resolve this issue, we basically need to swap out the required attributes (if any) and then would need to pin a lower-bound version on pennylane if I understand correctly.

[kessler-frost]

That is correct.

[kessler-frost]

Sorry @Andrew-S-Rosen , but I think this issue will be blocked by #1877 😓 . We don't currently have tests enabled for the qelectrons (which are the reason behind the Pennylane dependency), thus we cannot know whether upgrading to version 33 will break the feature or not (or if the feature has gotten broken due to a recent change yet).

[Andrew-S-Rosen]

Sounds good!!