Skip to content

[SCFD-5478][SCFD-5477][SCFD-5476][SCFD-5174] Velocity and Vorticity component #1085

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

Merged
merged 11 commits into from
May 28, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
66 changes: 57 additions & 9 deletions flow360/component/simulation/outputs/output_fields.py
Original file line number Diff line number Diff line change
Expand Up @@ -90,6 +90,9 @@
"pressure",
"vorticity",
"vorticityMagnitude",
"vorticity_x",
"vorticity_y",
"vorticity_z",
"wallDistance",
"numericalDissipationFactor",
"residualHeatSolver",
Expand Down Expand Up @@ -175,6 +178,13 @@
"mut",
"nuHat",
"vorticityMagnitude",
"vorticity_x",
"vorticity_y",
"vorticity_z",
"velocity_magnitude",
"velocity_x",
"velocity_y",
"velocity_z",
]

AllFieldNames = Literal[CommonFieldNames, SurfaceFieldNames, VolumeFieldNames, IsoSurfaceFieldNames]
Expand Down Expand Up @@ -253,23 +263,37 @@ def get_unit_for_field(field_name: str):
"pressure": {
"type": FIELD_TYPE_SCALAR,
},
"vorticity_x": {
"type": FIELD_TYPE_SCALAR,
},
"vorticity_y": {
"type": FIELD_TYPE_SCALAR,
},
"vorticity_z": {
"type": FIELD_TYPE_SCALAR,
},
}

# Predefined UDF expressions
PREDEFINED_UDF_EXPRESSIONS = {
"velocity": "velocity[0] = primitiveVars[1];"
+ "velocity[1] = primitiveVars[2];"
+ "velocity[2] = primitiveVars[3];",
"velocity": "velocity[0] = primitiveVars[1] * velocityScale;"
+ "velocity[1] = primitiveVars[2] * velocityScale;"
+ "velocity[2] = primitiveVars[3] * velocityScale;",
"velocity_magnitude": "double velocity[3];"
+ "velocity[0] = primitiveVars[1];"
+ "velocity[1] = primitiveVars[2];"
+ "velocity[2] = primitiveVars[3];"
+ "velocity_magnitude = magnitude(velocity);",
"velocity_x": "velocity_x = primitiveVars[1];",
"velocity_y": "velocity_y = primitiveVars[2];",
"velocity_z": "velocity_z = primitiveVars[3];",
"pressure": "pressure = primitiveVars[4];",
"wall_shear_stress_magnitude": "wall_shear_stress_magnitude = magnitude(wallShearStress);",
+ "velocity_magnitude = magnitude(velocity) * velocityScale;",
"velocity_x": "velocity_x = primitiveVars[1] * velocityScale;",
"velocity_y": "velocity_y = primitiveVars[2] * velocityScale;",
"velocity_z": "velocity_z = primitiveVars[3] * velocityScale;",
"pressure": "double gamma = 1.4;pressure = (usingLiquidAsMaterial) ? "
+ "(primitiveVars[4] - 1.0 / gamma) * (velocityScale * velocityScale) : primitiveVars[4];",
"wall_shear_stress_magnitude": "wall_shear_stress_magnitude = "
+ "magnitude(wallShearStress) * (velocityScale * velocityScale);",
"vorticity_x": "vorticity_x = (gradPrimitive[3][1] - gradPrimitive[2][2]) * velocityScale;",
"vorticity_y": "vorticity_y = (gradPrimitive[1][2] - gradPrimitive[3][0]) * velocityScale;",
"vorticity_z": "vorticity_z = (gradPrimitive[2][0] - gradPrimitive[1][1]) * velocityScale;",
}


Expand Down Expand Up @@ -377,3 +401,27 @@ def _distribute_shared_output_fields(solver_values: dict, item_names: str):
item.output_fields = []
if field not in item.output_fields:
item.output_fields.append(field)


def append_component_to_output_fields(output_fields: List[str]) -> List[str]:
"""
If "velocity" or "vorticity" is in the list, append their respective magnitude in output

Parameters:
-----------
output_fields : List[str]
The list of output fields to modify.

Returns:
--------
List[str]
The modified list of output fields with the component appended.
"""
output_fields_with_component = []
for field in output_fields:
output_fields_with_component.append(field)
if field == "velocity" and "velocity_magnitude" not in output_fields:
output_fields_with_component.append("velocity_magnitude")
if field == "vorticity" and "vorticityMagnitude" not in output_fields:
output_fields_with_component.append("vorticityMagnitude")
return output_fields_with_component
24 changes: 18 additions & 6 deletions flow360/component/simulation/translator/solver_translator.py
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,11 @@
PointArray,
PointArray2D,
)
from flow360.component.simulation.outputs.output_fields import generate_predefined_udf
from flow360.component.simulation.outputs.output_fields import (
PREDEFINED_UDF_EXPRESSIONS,
append_component_to_output_fields,
generate_predefined_udf,
)
from flow360.component.simulation.outputs.outputs import (
AeroAcousticOutput,
Isosurface,
Expand Down Expand Up @@ -232,7 +236,7 @@ def translate_output_fields(
],
):
"""Get output fields"""
return {"outputFields": output_model.output_fields.items}
return {"outputFields": append_component_to_output_fields(output_model.output_fields.items)}


def surface_probe_setting_translation_func(entity: SurfaceProbeOutput):
Expand Down Expand Up @@ -352,9 +356,11 @@ def translate_volume_output(
# Get outputFields
volume_output.update(
{
"outputFields": get_global_setting_from_first_instance(
output_params, volume_output_class, "output_fields"
).model_dump()["items"],
"outputFields": append_component_to_output_fields(
get_global_setting_from_first_instance(
output_params, volume_output_class, "output_fields"
).model_dump()["items"]
),
}
)
return volume_output
Expand Down Expand Up @@ -525,7 +531,13 @@ def process_output_fields_for_udf(input_params: SimulationParams):
if input_params.outputs:
for output in input_params.outputs:
if hasattr(output, "output_fields") and output.output_fields:
all_field_names.update(output.output_fields.items)
all_field_names.update(
append_component_to_output_fields(output.output_fields.items)
)
if isinstance(output, IsosurfaceOutput):
for isosurface in output.entities.items:
if isosurface.field in PREDEFINED_UDF_EXPRESSIONS:
all_field_names.add(isosurface.field)

if isinstance(input_params.operating_condition, LiquidOperatingCondition):
all_field_names.add("velocity_magnitude")
Expand Down
3 changes: 3 additions & 0 deletions tests/simulation/framework/test_unique_list.py
Original file line number Diff line number Diff line change
Expand Up @@ -114,6 +114,9 @@ def test_unique_list():
"'pressure', "
"'vorticity', "
"'vorticityMagnitude', "
"'vorticity_x', "
"'vorticity_y', "
"'vorticity_z', "
"'wallDistance', "
"'numericalDissipationFactor', "
"'residualHeatSolver', "
Expand Down
34 changes: 17 additions & 17 deletions tests/simulation/outputs/test_output_fields.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,9 @@ def test_generate_field_udf_no_unit(simulation_params):
result = generate_predefined_udf("velocity", simulation_params)

expected = (
"velocity[0] = primitiveVars[1];"
"velocity[1] = primitiveVars[2];"
"velocity[2] = primitiveVars[3];"
"velocity[0] = primitiveVars[1] * velocityScale;"
"velocity[1] = primitiveVars[2] * velocityScale;"
"velocity[2] = primitiveVars[3] * velocityScale;"
)

assert result == expected
Expand All @@ -38,9 +38,9 @@ def test_generate_field_udf_with_unit(simulation_params):

expected = (
"double velocity[3];"
"velocity[0] = primitiveVars[1];"
"velocity[1] = primitiveVars[2];"
"velocity[2] = primitiveVars[3];"
"velocity[0] = primitiveVars[1] * velocityScale;"
"velocity[1] = primitiveVars[2] * velocityScale;"
"velocity[2] = primitiveVars[3] * velocityScale;"
"velocity_m_per_s[0] = velocity[0] * 340.29400580821283;"
"velocity_m_per_s[1] = velocity[1] * 340.29400580821283;"
"velocity_m_per_s[2] = velocity[2] * 340.29400580821283;"
Expand All @@ -55,23 +55,23 @@ def test_generate_field_udf_velocity_components(simulation_params):
result = generate_predefined_udf("velocity_x_m_per_s", simulation_params)
expected = (
"double velocity_x;"
"velocity_x = primitiveVars[1];"
"velocity_x = primitiveVars[1] * velocityScale;"
"velocity_x_m_per_s = velocity_x * 340.29400580821283;"
)
assert result == expected

result = generate_predefined_udf("velocity_y_m_per_s", simulation_params)
expected = (
"double velocity_y;"
"velocity_y = primitiveVars[2];"
"velocity_y = primitiveVars[2] * velocityScale;"
"velocity_y_m_per_s = velocity_y * 340.29400580821283;"
)
assert result == expected

result = generate_predefined_udf("velocity_z_m_per_s", simulation_params)
expected = (
"double velocity_z;"
"velocity_z = primitiveVars[3];"
"velocity_z = primitiveVars[3] * velocityScale;"
"velocity_z_m_per_s = velocity_z * 340.29400580821283;"
)
assert result == expected
Expand All @@ -86,7 +86,7 @@ def test_generate_field_velocity_magnitude_no_unit(simulation_params):
"velocity[0] = primitiveVars[1];"
"velocity[1] = primitiveVars[2];"
"velocity[2] = primitiveVars[3];"
"velocity_magnitude = magnitude(velocity);"
"velocity_magnitude = magnitude(velocity) * velocityScale;"
)
assert result == expected

Expand All @@ -101,7 +101,7 @@ def test_generate_field_udf_velocity_magnitude(simulation_params):
"velocity[0] = primitiveVars[1];"
"velocity[1] = primitiveVars[2];"
"velocity[2] = primitiveVars[3];"
"velocity_magnitude = magnitude(velocity);"
"velocity_magnitude = magnitude(velocity) * velocityScale;"
"velocity_magnitude_m_per_s = velocity_magnitude * 340.29400580821283;"
)
assert result == expected
Expand All @@ -111,7 +111,7 @@ def test_generate_field_pressure_no_unit(simulation_params):
"""Test generating UDF expression for pressure fields."""

result = generate_predefined_udf("pressure", simulation_params)
expected = "pressure = primitiveVars[4];"
expected = "double gamma = 1.4;pressure = (usingLiquidAsMaterial) ? (primitiveVars[4] - 1.0 / gamma) * (velocityScale * velocityScale) : primitiveVars[4];"
assert result == expected


Expand All @@ -120,8 +120,8 @@ def test_generate_field_udf_pressure(simulation_params):

result = generate_predefined_udf("pressure_pa", simulation_params)
expected = (
"double pressure;"
"pressure = primitiveVars[4];"
"double pressure;double gamma = 1.4;"
"pressure = (usingLiquidAsMaterial) ? (primitiveVars[4] - 1.0 / gamma) * (velocityScale * velocityScale) : primitiveVars[4];"
"pressure_pa = pressure * 141855.012726525;"
)
assert result == expected
Expand All @@ -131,7 +131,7 @@ def test_genereate_field_wall_shear_stress_no_unit(simulation_params):
"""Test generating UDF expression for wall shear stress fields."""

result = generate_predefined_udf("wall_shear_stress_magnitude", simulation_params)
expected = "wall_shear_stress_magnitude = magnitude(wallShearStress);"
expected = "wall_shear_stress_magnitude = magnitude(wallShearStress) * (velocityScale * velocityScale);"
assert result == expected


Expand All @@ -141,7 +141,7 @@ def test_generate_field_udf_wall_shear_stress(simulation_params):
result = generate_predefined_udf("wall_shear_stress_magnitude_pa", simulation_params)
expected = (
"double wall_shear_stress_magnitude;"
"wall_shear_stress_magnitude = magnitude(wallShearStress);"
"wall_shear_stress_magnitude = magnitude(wallShearStress) * (velocityScale * velocityScale);"
"wall_shear_stress_magnitude_pa = wall_shear_stress_magnitude * 141855.012726525;"
)
assert result == expected
Expand All @@ -150,7 +150,7 @@ def test_generate_field_udf_wall_shear_stress(simulation_params):
def test_generate_field_udf_precedence(simulation_params):
"""Test that longer matching keys take precedence."""
result = generate_predefined_udf("velocity_x", simulation_params)
expected = "velocity_x = primitiveVars[1];"
expected = "velocity_x = primitiveVars[1] * velocityScale;"
assert result == expected


Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -166,6 +166,7 @@
"primitiveVars",
"residualNavierStokes",
"vorticity",
"vorticityMagnitude",
"solutionTurbulence",
"mutRatio",
"solutionTransition"
Expand All @@ -181,9 +182,9 @@
"v": "v",
"w": "w"
},
"userDefinedFields":[],
"userDefinedFields": [],
"usingLiquidAsMaterial": false,
"outputRescale": {
"velocityScale": 1.0
}
}
}
2 changes: 1 addition & 1 deletion tests/simulation/translator/ref/Flow360_liquid.json
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@
},
"userDefinedFields": [
{
"expression": "double velocity[3];velocity[0] = primitiveVars[1];velocity[1] = primitiveVars[2];velocity[2] = primitiveVars[3];velocity_magnitude = magnitude(velocity);",
"expression": "double velocity[3];velocity[0] = primitiveVars[1];velocity[1] = primitiveVars[2];velocity[2] = primitiveVars[3];velocity_magnitude = magnitude(velocity) * velocityScale;",
"name": "velocity_magnitude"
}
],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@
},
"userDefinedFields": [
{
"expression": "double velocity[3];velocity[0] = primitiveVars[1];velocity[1] = primitiveVars[2];velocity[2] = primitiveVars[3];velocity_magnitude = magnitude(velocity);",
"expression": "double velocity[3];velocity[0] = primitiveVars[1];velocity[1] = primitiveVars[2];velocity[2] = primitiveVars[3];velocity_magnitude = magnitude(velocity) * velocityScale;",
"name": "velocity_magnitude"
}
],
Expand Down
1 change: 1 addition & 0 deletions tests/simulation/translator/ref/Flow360_om6Wing.json
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@
"outputFields": [
"primitiveVars",
"vorticity",
"vorticityMagnitude",
"T",
"s",
"Cp",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@
"outputFields": [
"primitiveVars",
"vorticity",
"vorticityMagnitude",
"T",
"s",
"Cp",
Expand Down Expand Up @@ -189,4 +190,4 @@
"outputRescale": {
"velocityScale": 1.0
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,7 @@
"outputFields": [
"primitiveVars",
"vorticity",
"vorticityMagnitude",
"T",
"s",
"Cp",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,7 @@
"outputFields": [
"primitiveVars",
"vorticity",
"vorticityMagnitude",
"T",
"s",
"Cp",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@
"outputFields": [
"primitiveVars",
"vorticity",
"vorticityMagnitude",
"T",
"s",
"Cp",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@
"outputFields": [
"primitiveVars",
"vorticity",
"vorticityMagnitude",
"T",
"s",
"Cp",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@
"outputFields": [
"primitiveVars",
"vorticity",
"vorticityMagnitude",
"T",
"s",
"Cp",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@
"outputFields": [
"primitiveVars",
"vorticity",
"vorticityMagnitude",
"T",
"s",
"Cp",
Expand Down
Loading