I have two PRs for PennyLane-IonQ plugin that need review:
(1) This one contains some simple changes, mostly code cleanup: Add new IonQ forte-enterprise backend names to tests. by radumarg · Pull Request #132 · PennyLaneAI/PennyLane-IonQ · GitHub
(2) This one is more involved, adding support for IonQ ‘pauliexp’ gates:
Pauli exponential support by radumarg · Pull Request #134 · PennyLaneAI/PennyLane-IonQ · GitHub
How can I get these two PRs reviewed? Thank you!
Hi @Radu_Marginean, I’m sorry we missed your PRs!
We don’t look at that repo too often since it’s usually very quiet.
I’ll check with the team to see if they’re available. Are you part of the IonQ team? Or is this blocking work for research?
Also, I don’t know if our team will be able to test this on the Aria backend. Will you have access to test this if needed?
Hi @CatalinaAlbornoz, I am not part of IonQ team but this work was done at the request of IonQ. Spencer Churchill from IonQ team (@splch on Github) should be included in the communication for these two tickets.
The first ticket does not need much extra testing I believe. The second ticket already has tests using the online IonQ simulator, please see those test using “requires_api”. So while probably testing against Aria is not strictly necessary, such test can be performed if deemed necessary during the code review.
Thank you!
Thanks for this info @Radu_Marginean!
Our team will take a look at the PRs and determine if it needs some deeper review or if they’re mostly good to go. It may take a couple of days before they have an answer but it shouldn’t take more than a week. I’ll keep you updated on any news or progress. Thanks again for opening these PRs!