Skip to content

Silo memory leak workaround #450

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
Merged
ldowen merged 6 commits into from
Dec 17, 2025
Merged

Silo memory leak workaround #450

ldowen merged 6 commits into from
Dec 17, 2025

Conversation

@ldowen
Copy link
Collaborator

@ldowen ldowen commented Dec 16, 2025 •
edited
Loading

Summary

  • This PR is a workaround for a memory leak with our Silo python wrappers.
  • It does the following:
    • Adds Silo python wrappers to our Silo install and python virtual environment, though they are currently unused.
    • Reduces the memory impact of the leak by 100x.

ToDo :

  • Annotate RELEASE_NOTES.md with notable changes.
  • Create LLNLSpheral PR pointing at this branch. (PR#190)
  • LLNLSpheral PR has passed all tests.

@ldowen ldowen requested a review from jmikeowen December 16, 2025 23:44
@ldowen ldowen self-assigned this Dec 16, 2025
@ldowen ldowen linked an issue Dec 16, 2025 that may be closed by this pull request
Silo memory leak #416
Closed
@ldowen ldowen mentioned this pull request Dec 16, 2025
Closed
@ldowen ldowen merged commit 8e54964 into develop Dec 17, 2025
2 checks passed
@ldowen ldowen deleted the 416-silo-memory-leak branch December 17, 2025 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jmikeowen jmikeowen jmikeowen approved these changes

Assignees

@ldowen ldowen

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Silo memory leak

3 participants

@ldowen @jmikeowen