Skip to content
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

Valgrind error in SobolReporter #30141

Closed
zachmprince opened this issue Mar 19, 2025 · 0 comments · Fixed by #30142
Closed

Valgrind error in SobolReporter #30141

zachmprince opened this issue Mar 19, 2025 · 0 comments · Fixed by #30142
Assignees
@zachmprince
Labels
C: Modules/Stochastic Tools Tickets pertaining to the stochastic_tools module P: normal A defect affecting operation with a low possibility of significantly affects. T: defect An anomaly, which is anything that deviates from expectations.

Comments

@zachmprince
Copy link
Contributor

Bug Description

The STM workshop examples from #30007 uncovered a memory leak in the SobolReporter for cases where the matrices aren't resampled. See https://civet.inl.gov/job/2754167/

Steps to Reproduce

cd modules/stochastic-tools
./run_tests --re step4 --valgrind --oprof

Impact

Next is broken and there might be undefined behavior in that object.

[Optional] Diagnostics

No response
@zachmprince zachmprince added C: Modules/Stochastic Tools Tickets pertaining to the stochastic_tools module P: normal A defect affecting operation with a low possibility of significantly affects. T: defect An anomaly, which is anything that deviates from expectations. labels Mar 19, 2025
@zachmprince zachmprince self-assigned this Mar 19, 2025
zachmprince added a commit to zachmprince/moose that referenced this issue Mar 19, 2025
@zachmprince
Fixing valgrind failure in STM idaholab#30141
1f1a3c3
zachmprince added a commit to zachmprince/moose that referenced this issue Mar 19, 2025
@zachmprince
Making a new, smaller, test that replicates the valgrind error idahol…
94572fa 
…ab#30141
zachmprince added a commit to zachmprince/moose that referenced this issue Mar 19, 2025
@zachmprince
Renaming STM workshop examples test spec so they aren't run by defaul…
a977d32 
…t in the TestHarness idaholab#30141
@zachmprince zachmprince mentioned this issue Mar 19, 2025
Merged
tophmatthews pushed a commit to tophmatthews/moose that referenced this issue Mar 21, 2025
@zachmprince @tophmatthews
Fixing valgrind failure in STM idaholab#30141
ec5b5d9
tophmatthews pushed a commit to tophmatthews/moose that referenced this issue Mar 21, 2025
@zachmprince @tophmatthews
Making a new, smaller, test that replicates the valgrind error idahol…
69f4188 
…ab#30141
tophmatthews pushed a commit to tophmatthews/moose that referenced this issue Mar 21, 2025
@zachmprince @tophmatthews
Renaming STM workshop examples test spec so they aren't run by defaul…
f2722c8 
…t in the TestHarness idaholab#30141
maxnezdyur pushed a commit to maxnezdyur/moose that referenced this issue Mar 25, 2025
@zachmprince @maxnezdyur
Fixing valgrind failure in STM idaholab#30141
2636cd0
maxnezdyur pushed a commit to maxnezdyur/moose that referenced this issue Mar 25, 2025
@zachmprince @maxnezdyur
Making a new, smaller, test that replicates the valgrind error idahol…
306915b 
…ab#30141
maxnezdyur pushed a commit to maxnezdyur/moose that referenced this issue Mar 25, 2025
@zachmprince @maxnezdyur
Renaming STM workshop examples test spec so they aren't run by defaul…
fe744d2 
…t in the TestHarness idaholab#30141
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zachmprince zachmprince

Labels
C: Modules/Stochastic Tools Tickets pertaining to the stochastic_tools module P: normal A defect affecting operation with a low possibility of significantly affects. T: defect An anomaly, which is anything that deviates from expectations.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix valgrind failure in STM zachmprince/moose
1 participant
@zachmprince