Removing the set_resolutions option to resolve mandatory receptor issues

Exawind / nalu-wind

Solver for wind farm simulations targeting exascale computational platforms

https://nalu-wind.readthedocs.io

Other

122 stars 83 forks source link

Removing the set_resolutions option to resolve mandatory receptor issues #1267

Closed itopcuoglu closed 2 months ago

psakievich commented 2 months ago

Are you sure this is never needed?

itopcuoglu commented 2 months ago

None of the past regression tests use the set_resolutions: no option. We could also leave it in place, and make the code display a warning to say something like "WARNING: Nodal resolutions calculated by Tioga are not communicated at shared nodes. Use the default option "set_resolutions: yes" to ensure consistency at shared nodes".

psakievich commented 2 months ago

Seems reasonable to me then.