PennyLaneAI / PennyLaneAI/catalyst
Add EnumAttr to the Gradient Dialect for different differentiation methods.
Open
Nobody has claimed this yet.
chore
compiler
good first issue
- Dominant language
- Python
- Stars
- 234
- Forks
- 84
- Avg merge
- 2d 15h
- Merged PRs (30d)
- 66
Description
Issue description
- Actual behavior: Right now, the gradient method is a StringAttr, but it would be better to have an EnumAttr.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the Gradient Dialect and the current gradient method StringAttr. Review how its differentiation methods are represented, then define the EnumAttr coverage and verify that the dialect accepts the intended methods without relying on arbitrary strings.
Written by the indexing model from the issue text.
Assessment
- Domain
- compilers
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100