load_node(1) != load_node(1) ... The node should be equal to itself.

[yakutovicha]

Describe the bug

In [1]: load_node(1) == load_node(1)
Out[1]: False

Expected behaviour

In [1]: load_node(1) == load_node(1)
Out[1]: True

[greschd]

I think this is a duplicate of #1917.

Note that there's some argument there about what exactly the equality operator should do, but I think we agree that the same nodes should compare equal.

[yakutovicha]

heh, sorry, I was searching for some words like 'equal', but didn't find the issue. Thanks @greschd

[greschd]

Yeah no problem, it's good that we good a ping about fixing that issue.

Since we were looking for broader feedback (see this comment), would be great to also hear your opinion on that.

[yakutovicha]

Yeah no problem, it's good that we good a ping about fixing that issue.

Since we were looking for broader feedback (see this comment), would be great to also hear your opinion on that.

I am currently going through. And I feel exactly as @giovannipizzi wrote (c):

I have to admit that while reading it, at every comment I thought "ok we should do this" and at the following "ah right, wait!"...