Alex Wissmann
Alex Wissmann
No there is no reason anymore. The only change I've left in is moving the cancel from Eval to Session.run(), since it cleans up the call to maintainSliceGroup.
> Why touch anything related to the contexts at all? Is it fixing some issue? Just a refactor, trying to leave the code a bit nicer than I found it...
Thanks for the explanation. I've removed those changes.
This is much cleaner, thank you. I've just pushed these changes.
With the code as is we will see the machine lost logs on shutdown immediately, meaning users will see this more often now. Should we do something else to hide...