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

test return types for frule and rrule #50

Open
CarloLucibello opened this issue Jul 11, 2020 · 3 comments
Open

test return types for frule and rrule #50

CarloLucibello opened this issue Jul 11, 2020 · 3 comments
Labels
enhancement New feature or request question Further information is requested

Comments

@CarloLucibello
Copy link

CarloLucibello commented Jul 11, 2020

See JuliaDiff/ChainRules.jl#233 and reference therein for possible issues when unexpected casts happen

@nickrobinson251
Copy link
Contributor

it'd be good to add a little more detail here :) For example, does this in anyway require JuliaDiff/FiniteDifferences.jl#97, @willtebbutt ?

@nickrobinson251 nickrobinson251 added enhancement New feature or request question Further information is requested labels Aug 4, 2020
@willtebbutt
Copy link
Member

willtebbutt commented Aug 6, 2020

For example, does this in anyway require JuliaDiff/FiniteDifferences.jl#97

Hmm this seems like a separate issue from checking the numerical correctness of the gradients. Rather, this would require some functionality to say whether a particular "differential" is valid for a particular primal.

I don't think we'll be able to do this with 100% accuracy, but we should be able to catch it in a lot of cases. So you could imagine this taking the form of a passing default implementation, with methods implemented to catch specific cases that we know / care about - e.g. to check for the casting behaviour that @CarloLucibello pointed out / not representing the differential of a Float64 with a ComplexF64 or whatever.

This relates to conversations like:

(232 probably being the main conversation around this now)

@nickrobinson251
Copy link
Contributor

also JuliaDiff/ChainRulesCore.jl#106

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants