Skip to content

Conversation

ADH-LukeBollam
Copy link
Contributor

Fixes the Session to not immediately dispose the graph.

When running the model there appears to still be a memory leak (though much smaller). Checking the stack after the test has completed shows some tensorflow ops and a graph still in memory:

image

@ADH-LukeBollam
Copy link
Contributor Author

@Oceania2018 even if the memory leak isn't fixed completely, could we get this merged in? It's still a substantial reduction in memory growth compared to the last release with a working SavedModel

@Oceania2018
Copy link
Member

@LukeBolly Can you resolve the conflicts?

@ADH-LukeBollam
Copy link
Contributor Author

ADH-LukeBollam commented Oct 6, 2021

Hi @Oceania2018 , do you have an estimate on when the next release will be that contains this PR?

@Oceania2018
Copy link
Member

@LukeBolly will release this week.

@Oceania2018 Oceania2018 merged commit 3feff99 into SciSharp:master Oct 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants