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
Rewrites can be optionally included or excluded based on their registered names. However, some of the registered names don't match the name of the rewrite function, which is shown with optimizer_verbose=True
...
rewriting: rewrite push_out_seq_scan replaces for{cpu,scan_fn}.0 of for{cpu,scan_fn}(Subtensor{int64}.0, Subtensor{:int64:}.0, Subtensor{int64}.0) with Elemwise{exp,no_inplace}.0 of Elemwise{exp,no_inplace}(Subtensor{:int64:}.0)
...
You would think you can disable it, by excluding push_out_seq_scan, but that does not work
...
rewriting: rewrite push_out_seq_scan replaces for{cpu,scan_fn}.0 of for{cpu,scan_fn}(Subtensor{int64}.0, Subtensor{:int64:}.0, Subtensor{int64}.0) with Elemwise{exp,no_inplace}.0 of Elemwise{exp,no_inplace}(Subtensor{:int64:}.0)
...
Still there. Because it's actually registered with the name scan_pushout_seqs_ops:
We should register the rewrites with the same name of the function that is shown in the debug (easy) or find a way to show the registration name instead (possibly harder?).
The text was updated successfully, but these errors were encountered:
ricardoV94
changed the title
Don't create new rewrite names and tags
Don't customize rewrite names
Mar 13, 2023
Rewrites can be optionally included or excluded based on their registered names. However, some of the registered names don't match the name of the rewrite function, which is shown with
optimizer_verbose=True
For instance:
You would think you can disable it, by excluding
push_out_seq_scan
, but that does not workStill there. Because it's actually registered with the name
scan_pushout_seqs_ops
:pytensor/pytensor/scan/rewriting.py
Line 2486 in 277559b
Now it's excluded!
We should register the rewrites with the same name of the function that is shown in the debug (easy) or find a way to show the registration name instead (possibly harder?).
The text was updated successfully, but these errors were encountered: