You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
AD graph transformations, such as canonicalize_graph(gm: GraphModule, ...) -> GraphModule have a signature that implies that they return a transformed graph instance while leaving the gm unchanged. These transformations actually mutate the GraphModule instance in-place, which can be confusing.
Change all transform_graph(gm: GraphModule, ...) -> GraphModule to transform_graph(gm: GraphModule, ...) -> None and fix the code accordingly.