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.

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

LineMaterialSamplerBase does not set subdomain ID #16959

Closed
joshuahansel opened this issue Feb 11, 2021 · 0 comments · Fixed by #16960
Closed

LineMaterialSamplerBase does not set subdomain ID #16959

joshuahansel opened this issue Feb 11, 2021 · 0 comments · Fixed by #16960
Labels
C: Framework T: defect An anomaly, which is anything that deviates from expectations.

Comments

@joshuahansel
Copy link
Contributor

Bug Description

In the element loop in LineMaterialSamplerBase<T>::execute(), the subdomain ID is not set before _subproblem.prepare(elem, _tid), leading to a failed assertion in Assembly::reinit(elem) that _current_subdomain_id == _current_elem->subdomain_id().

Steps to Reproduce

Run the following input file in debug mode and observe the assertion failure:
wrong_subdomain_id.txt

Impact

Inability to use LineMaterialRealSampler with multiple subdomains.

@joshuahansel joshuahansel added C: Framework T: defect An anomaly, which is anything that deviates from expectations. labels Feb 11, 2021
joshuahansel added a commit to joshuahansel/moose that referenced this issue Feb 11, 2021
joshuahansel added a commit to joshuahansel/moose that referenced this issue Feb 11, 2021
joshuahansel added a commit to joshuahansel/moose that referenced this issue Feb 11, 2021
jain651 pushed a commit to jain651/moose that referenced this issue Apr 19, 2021
aeslaughter pushed a commit to aeslaughter/moose that referenced this issue Jun 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: Framework T: defect An anomaly, which is anything that deviates from expectations.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant