dfdx icon indicating copy to clipboard operation
dfdx copied to clipboard

#67 `Optimizer::update` now returns `Result<(), UnusedParamsError>`

Open coreylowman opened this issue 1 year ago • 2 comments

Closes #67

coreylowman avatar Jul 20 '22 21:07 coreylowman

This is now a breaking change and CanUpdateWithGradients is now quite a bit harder to read. BUT now users can decide if unused parameters are an issue, and if so get a list of all of them. Otherwise all used parameters are updated

coreylowman avatar Jul 20 '22 22:07 coreylowman

TODO does this affect performance at all?

coreylowman avatar Jul 25 '22 16:07 coreylowman