pytensor icon indicating copy to clipboard operation
pytensor copied to clipboard

Remove `list_of_nodes` in favor of similar `applys_between`

Open ferrine opened this issue 3 years ago • 3 comments

Description

https://github.com/pymc-devs/pytensor/blob/main/pytensor/graph/basic.py#L951 and https://github.com/pymc-devs/pytensor/blob/main/pytensor/graph/basic.py#L1589

ferrine avatar Nov 30 '22 07:11 ferrine

The second link is pointing to a different function than the one you mentioned list_of_nodes

ricardoV94 avatar Nov 30 '22 09:11 ricardoV94

updated

ferrine avatar Nov 30 '22 14:11 ferrine

list_of_nodes is only used in one place and I think it can be safely replaced by the applys_between

ricardoV94 avatar Dec 07 '23 12:12 ricardoV94