NormalizingFlows.jl icon indicating copy to clipboard operation
NormalizingFlows.jl copied to clipboard

Enzyme extension unnecessarily computing original result

Open wsmoses opened this issue 6 months ago • 0 comments

https://github.com/TuringLang/NormalizingFlows.jl/blob/58d3e57e66872274a1b6f46ffe68fc54da642d90/ext/NormalizingFlowsEnzymeExt.jl#L21

This line can be weakened from ReverseWithPrimal to just Reverse, since you don't use the result.

Moreover the issue referenced in the comments above have been closed/resolved

wsmoses avatar Aug 07 '24 04:08 wsmoses