David Ittah
David Ittah
Hi @ndcroos, welcome to Catalyst and thank you for contributing! 🙂 The Catalyst team will be taking a look at your PR shortly. In general we use `black` and `isort`...
Hi @ndcroos, I think your change in [7009853](https://github.com/PennyLaneAI/catalyst/pull/263/commits/7009853e68c54bee093c31e6f5d51724fe6838f2) fixed it. Because the given gate / unitary itself requires multiple parameters (in a list or array), you just needed an additional...
Hi @ndcroos, you might need to revert your latest commit [a155b73](https://github.com/PennyLaneAI/catalyst/pull/263/commits/a155b73f821e88b4e30e1eb6a836615de2fcf078) as it seems to have changed the versions of our dependencies with respect to main.
Hi @ndcroos, I wanted to check in to see if you were still interested in contributing these tests? :)
Thank you for working on this PR! I'll close it as stale now but feel free to reopen at any time if you wish to continue working on it :)
> > @mehrdad2m you are also merging some commits from rc to main, this is intended, right? > > yes, I originally made my PR based on the release branch...
Briefly looked into this issue, and the root cause is that the ZNE wrapper only works directly on QNodes, but the dynamic_one_shot transform replaces a QNode with a classical function...
@dime10 Check if fixed
Can confirm this is now fixed, test here: https://github.com/PennyLaneAI/catalyst/blob/f34646ed1268cda674fdd6ed336f5deed0bd10da/frontend/test/lit/test_mitigation.py#L28
@paul0403 It looks like a meta data file is needed so we can look at the built version :)