-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add SkipOptimisation #243
Comments
We actually do this quite a bit already. See here. It might make sense for me to split this out into a separate file though, so that it's more clearly labelled, rather than just being under "misc". |
That's not critical as long as we have explicit comments on the purpose in the file. |
Sounds good. I've just reviewed the comment associated to them, and it's quite outdated. I'll make updating / improving it part of this issue. |
We could optimise functions that we know won't influence gradients. Some examples from ReverseDiff:
Any others?
The text was updated successfully, but these errors were encountered: