Rename argnum with argnums in Catalyst grad related functions

PennyLaneAI / pennylane

PennyLane is a cross-platform Python library for quantum computing, quantum machine learning, and quantum chemistry. Train a quantum computer the same way as a neural network.

https://pennylane.ai

Apache License 2.0

2.36k stars 604 forks source link

Rename argnum with argnums in Catalyst grad related functions #6117

Closed rmoyard closed 2 months ago

rmoyard commented 3 months ago

Context: Catalyst is renaming argnum with argnums. https://github.com/PennyLaneAI/catalyst/pull/1036

Description of the Change: Update calls to Catalyst functions.

github-actions[bot] commented 3 months ago

Hello. You may have forgotten to update the changelog! Please edit doc/releases/changelog-dev.md with:

A one-to-two sentence description of the change. You may include a small working example for new features.
A link back to this PR.
Your name (or GitHub username) in the contributors section.

rmoyard commented 2 months ago

@josh146

I will update the Catalyst version to use the one from test pypi of tomorrow
I will merge the Catalyst one first and then this one because we need to install Catalyst
Tag @mudit2812 and @rauletorresc for visibility

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.65%. Comparing base (1f55c88) to head (e9cb0a4). Report is 9 commits behind head on master.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #6117 +/- ## ========================================== - Coverage 99.67% 99.65% -0.02% ========================================== Files 432 432 Lines 41839 41696 -143 ========================================== - Hits 41702 41554 -148 - Misses 137 142 +5 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.