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

Document composite:minmax_per_element_operator #366

Merged
merged 9 commits into from
Oct 15, 2023
Merged

Document composite:minmax_per_element_operator #366

Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
ec05643
Fix broken link, and port changes from 0.3.1 back to develop
roosre Sep 14, 2023
6a0bbbd
Merge branch 'main' of https://github.com/ansys/pydpf-composites
roosre Sep 21, 2023
d9fc98b
Merge branch 'main' of https://github.com/ansys/pydpf-composites
roosre Sep 27, 2023
18dd8b2
Merge branch 'main' of https://github.com/ansys/pydpf-composites
roosre Oct 13, 2023
b942916
Add note that the critical layer indices are 1-based.
roosre Oct 13, 2023
dad47a4
add note when using get_selected_indices
roosre Oct 13, 2023
440be83
update comments and note about 1-based indices
roosre Oct 15, 2023
98251bd
implement remark from the code review.
roosre Oct 15, 2023
1cf6637
improve documentation
roosre Oct 15, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions examples/010_harmonic_example.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -131,6 +131,8 @@
failure_evaluator.inputs.strains_container(strain_at_phase)
failure_evaluator.inputs.mesh(composite_model.get_mesh())

# Note: the min/max layer indices are 1-based since
# WB 2024 R1 (server version 7 and above)
roosre marked this conversation as resolved.
Show resolved Hide resolved
minmax_per_element = dpf.Operator("composite::minmax_per_element_operator")
minmax_per_element.inputs.fields_container(failure_evaluator)
minmax_per_element.inputs.mesh(composite_model.get_mesh())
Expand Down
2 changes: 2 additions & 0 deletions src/ansys/dpf/composites/_composite_model_impl.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -331,6 +331,8 @@ def evaluate_failure_criteria(
# Ensure that sandwich criteria are evaluated
evaluate_failure_criterion_per_scope_op.inputs.request_sandwich_results(True)

# Note: the min/max layer indices are 1-based since
# WB 2024 R1 (server version 7 and above)
minmax_el_op = dpf.Operator("composite::minmax_per_element_operator")
minmax_el_op.inputs.fields_container(
evaluate_failure_criterion_per_scope_op.outputs.failure_container
Expand Down
4 changes: 4 additions & 0 deletions src/ansys/dpf/composites/select_indices.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,10 @@ def get_selected_indices(
Notes
-----
Returns an empty selection if any of the collections is empty.

The indices (nodes, layers and spots) are 0-based. Pay attention to this
roosre marked this conversation as resolved.
Show resolved Hide resolved
when using the "composite::minmax_per_element_operator" where the
min/max layer indices are 1-based.
"""
if layers is None:
layer_indices: Collection[int] = range(element_info.n_layers)
Expand Down
2 changes: 2 additions & 0 deletions tests/basic_workflow_test.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -87,6 +87,8 @@ def test_basic_workflow(dpf_server, distributed_rst):
failure_evaluator.inputs.stresses_container(stress_operator.outputs.fields_container)
failure_evaluator.inputs.mesh(mesh_provider.outputs.mesh)

# Note: the min/max layer indices are 1-based since
# WB 2024 R1 (server version 7 and above)
minmax_per_element = dpf.Operator("composite::minmax_per_element_operator")
minmax_per_element.inputs.fields_container(failure_evaluator.outputs.fields_container)
minmax_per_element.inputs.mesh(mesh_provider.outputs.mesh)
Expand Down
Loading