-
-
Notifications
You must be signed in to change notification settings - Fork 38
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
gradient throws error #92
Comments
What's this? I have no idea what you're trying to discuss here. |
The README mentions applications through automatic differentiation, but there seem to be some issues with it. Some examples: using Grassman, Zygote
basis"3" # this defines (⟨×××⟩, v, v₁, v₂, v₃, v₁₂, v₁₃, v₂₃, v₁₂₃)
Zygote.gradient(v->v^2 ,v₁) # this gives (2v₁,)
Zygote.gradient(v -> 1 * v^2, v₁) # similar error as above The last two calls should be identical but are not. To be fair, the issue seems to be in DirectSum/AbstractTensors, so I can move the issue to there if you like, though iirc both of those are also your packages. |
Grassmann.jl has its own experimental automatic differentiation built-in to the tensor algebra. It is not intended to be used with a separate AD package, but is its own experimental AD design from scratch based on tensor algebra. I don't know about Zygote syntax. |
If you want to discuss my experimental AD design, we can do that in private. I no longer like to publicly discuss design details, these type of discussions with me are now a premium feature for private discussion. |
interesting. are there any examples of its usage? fwiw, adding zygote support would increase usability of the package a fair bit for several applications, and i'd be willing to chip in some money for it. |
Alright, I will email you to discuss this privately for now. |
By the way, you can make all your examples above work just by using a julia> @basis S"++"
(⟨++⟩, v, v₁, v₂, v₁₂)
julia> Zygote.gradient(v -> 1 * v^2, v₁) # similar error as above
(2v₁,)
julia> Zygote.gradient(v->0v, v1)
(0v,) |
v1 is the standard basis vector
The text was updated successfully, but these errors were encountered: