Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Implement Apply.__eq__ #171

Open
brandonwillard opened this issue Nov 16, 2020 · 0 comments
Open

Implement Apply.__eq__ #171

brandonwillard opened this issue Nov 16, 2020 · 0 comments
Labels
enhancement New feature or request

Comments

@brandonwillard
Copy link
Member

There is no __eq__ implementation for Apply nodes, but there probably should be if we want to fix the unnecessary, confusing, and extremely error-prone in-place updates that are currently performed on Theano graphs.

This is one step in #37.
@brandonwillard brandonwillard added the enhancement New feature or request label Nov 16, 2020
@michaelosthege michaelosthege added this to the 2.0 milestone Dec 15, 2020
@brandonwillard brandonwillard removed this from the 2.x milestone Jan 5, 2022
@brandonwillard brandonwillard changed the title Implement Apply.__eq__ Implement Apply.__eq__ Jan 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@brandonwillard @michaelosthege