Our decoration naming and usage is rather messy, mostly because we rewrote it several times. The most glaring part is where we have a single decoration ForwardDerivativeDecoration to hold user-provided forward derivatives but two UserDefinedBackwardDerivativeDecoration and BackwardDerivativeDecoration, for the reverse mode.
We should simply be able to remove UserDefinedBackwardDerivativeDecoration and cut down on code duplication & confusion.
Our decoration naming and usage is rather messy, mostly because we rewrote it several times. The most glaring part is where we have a single decoration
ForwardDerivativeDecorationto hold user-provided forward derivatives but twoUserDefinedBackwardDerivativeDecorationandBackwardDerivativeDecoration, for the reverse mode. We should simply be able to removeUserDefinedBackwardDerivativeDecorationand cut down on code duplication & confusion.