Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix AutoEnzyme docstring for constant_function #73

Merged
merged 3 commits into from
Jul 18, 2024

Conversation

gdalle
Copy link
Collaborator

@gdalle gdalle commented Jul 17, 2024

There was a typo in #72 for the docstring, I switched true and false in the explanation of constant_function

@gdalle
Copy link
Collaborator Author

gdalle commented Jul 17, 2024

@willtebbutt I wonder if the same kind of distinction would make sense for Tapir? To clarify, we're introducing this for Enzyme so that DifferentiationInterface can know whether to Duplicate/shadow the function f itself or not.

src/dense.jl Outdated Show resolved Hide resolved
@ChrisRackauckas ChrisRackauckas merged commit 024ac94 into main Jul 18, 2024
4 checks passed
@ChrisRackauckas ChrisRackauckas deleted the gd/fix_enzyme_doc branch July 18, 2024 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants