modulus-sym icon indicating copy to clipboard operation
modulus-sym copied to clipboard

🐛[BUG]: dimensions in Chip 2D example

Open shourya-p-otta opened this issue 1 year ago • 2 comments

Version

23.09,23.11, 24.01

On which installation method(s) does this occur?

No response

Describe the issue

nd_copper_source_grad = copper_heat_flux * length_scale / temp_scale is not non-dimensional. Looks like final units are W/m^2/m-K = kg m^2 / s^3 /m^2 /m-K = kg / s^3 /m-K. This needs to be fixed.

https://github.com/NVIDIA/modulus-sym/blob/dc5c0f759bb19e710fae0ba70cda63e129102774/examples/chip_2d/chip_2d_solid_fluid_heat_transfer_heat.py#L115

Minimum reproducible example

No response

Relevant log output

No response

Environment details

No response

Other/Misc.

No response

shourya-p-otta avatar Mar 04 '24 17:03 shourya-p-otta

@shourya-p-otta please refer this issue here: https://github.com/NVIDIA/modulus-sym/issues/109. This was a typo that was addressed recently

ktangsali avatar Mar 04 '24 17:03 ktangsali

@ktangsali the results need to be updated as well in the documentation.

shourya-p-otta avatar Mar 04 '24 18:03 shourya-p-otta

I verified, and the typo does not show up in the docs. Closing as the original bug is fixed. Please re-open if the issue persists.

ktangsali avatar Apr 11 '25 01:04 ktangsali