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

Allow ElementMaterialSampler to accept material property names #29705

Open
jessecarterMOOSE opened this issue Jan 18, 2025 · 0 comments · May be fixed by #29706
Open

Allow ElementMaterialSampler to accept material property names #29705

jessecarterMOOSE opened this issue Jan 18, 2025 · 0 comments · May be fixed by #29706
Labels
T: task An enhancement to the software.

Comments

@jessecarterMOOSE
Copy link
Contributor

Motivation

Currently ElementMaterialSampler only takes a single Material name as an input and returns all its declared properties. Need a way handling multiple Materials.

Design

I would like to specify the material properties directly.

Impact

The object would take either a Material name or a property name(s).

I have a branch for this and will put up a PR shortly
@jessecarterMOOSE jessecarterMOOSE added the T: task An enhancement to the software. label Jan 18, 2025
jessecarterMOOSE added a commit to jessecarterMOOSE/moose that referenced this issue Jan 18, 2025
@jessecarterMOOSE
allow material property names to be passed to ElementMaterialSampler, c…
6c30973 
…loses idaholab#29705
jessecarterMOOSE added a commit to jessecarterMOOSE/moose that referenced this issue Jan 18, 2025
@jessecarterMOOSE
allow material property names to be passed to ElementMaterialSampler, c…
b65eb70 
…loses idaholab#29705
@jessecarterMOOSE jessecarterMOOSE linked a pull request Jan 18, 2025 that will close this issue
Open
jessecarterMOOSE added a commit to jessecarterMOOSE/moose that referenced this issue Jan 18, 2025
@jessecarterMOOSE
apply style changes, closes idaholab#29705
427233e
jessecarterMOOSE added a commit to jessecarterMOOSE/moose that referenced this issue Jan 21, 2025
@jessecarterMOOSE
disable mesh renumbering so distributed mesh tests don't diff, ref #i…
0ab3a0e 
…daholab#29705
jessecarterMOOSE added a commit to jessecarterMOOSE/moose that referenced this issue Jan 21, 2025
@jessecarterMOOSE
test for failure when sampling property that does not exist on block,…
b1c939f 
… ref idaholab#29705
jessecarterMOOSE added a commit to jessecarterMOOSE/moose that referenced this issue Jan 21, 2025
@jessecarterMOOSE
address review comments, ref idaholab#29705
34a9705
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
T: task An enhancement to the software.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allow material property names to be used in ElementMaterialSampler jessecarterMOOSE/moose
1 participant
@jessecarterMOOSE