Fix a few crossrefs + update Zygote's page #2064
Conversation
docs/src/models/losses.md
Outdated
| @@ -21,7 +21,7 @@ loss(ŷ, y, agg=x->mean(w .* x)) # weighted mean | |||
| loss(ŷ, y, agg=identity) # no aggregation. | |||
| ``` | |||
|
|
|||
| ### Function listing | |||
| ## Function listing | |||
There was a problem hiding this comment.
My thinking with H3 was that it's nice to separate the listing from the introductory blurb, but it doesn't really need an entry on the side-bar if there's just one heading.
docs/src/training/zygote.md
Outdated
| Zygote.diaghessian | ||
| Zygote.withjacobian | ||
| Zygote.withgradient |
There was a problem hiding this comment.
I think my vote would be to include withgradient alongside gradient. Then this section would just be about second derivatives, so it can have a heading more useful than "utilities".
However, these are very limited, Zygote over Zygote never works that well. So perhaps it needs disclaimers here.
And perhaps it wants explanation of how to use hessian with destructure, since oherwise it can't act on a Flux model parameters?
There was a problem hiding this comment.
(Such a Hessian example is now in destructure's own page)
docs/src/training/zygote.md
Outdated
| Zygote.diaghessian | ||
| Zygote.withjacobian | ||
| Zygote.withgradient |
There was a problem hiding this comment.
(Such a Hessian example is now in destructure's own page)
|
Yes, went through the page and decided to remove this section! |
h2(##) does not appear in the sidebarPR Checklist