Closed Xxy1016 closed 1 year ago
Hi! Thanks for this, if you could sign the CLA I'd be happy to merge it :+1:
Hi, @OskarDamkjaer, thanks for your attention. I have emailed my signed CLA as instruction in here. Is there any thing I should do before getting this merged?
@Xxy1016 Thanks! I'll rerun our tests and merge when they come back green!
When we reset a Graph, we see that the
expandedNodeMap
was not reset. We think we should clear all instance variables while resetting the graph. So this PR adds the reset to theexpandedNodeMap