Consider replacing `SharedVariable.default_update` with in-graph `Op`s
brandonwillard opened this issue · comments
Brandon T. Willard commented
We could possibly replace SharedVariable.default_update
s with an Op
—call it Update
—that takes a SharedVariable
and its update graph as inputs.
Such an approach would reduce the many confusions arising from this out-of-graph attribute, and make it easier to keep updates consistent with any replacements that are intended to affect them (e.g. when performing replacements outside of the aesara.function
compilation context).