This repository was archived by the owner on Nov 17, 2025. It is now read-only.
Remove Type.filter_inplace
#1248
markusschmaus
started this conversation in
General
Replies: 1 comment
-
|
Yeah, there's still a bit of left over machinery from Theano, and that's a good example. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Type.filter_inplaceis currently not implemented anywhere in the code and the only place where it is used theNotImplementedErroris caught andType.filteris used instead. Based on the comment it seems that this is some leftover from some earlier feature. Is there any reason why this should be kept around? Otherwise I am going to remove it in #1207.Beta Was this translation helpful? Give feedback.
All reactions