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.

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

Halo exchange possibly needed during seaice_init_berg_velocity_solver #10

Open
darincomeau opened this issue Jul 2, 2019 · 0 comments

Comments

@darincomeau
Copy link
Owner

There may be a bug that when non-zero namelist options are used for:
config_berg_initial_uvelocity_geo
config_berg_initial_vvelocity_geo
that they may not be used correctly due to a missing halo exchange. This arose during testing of small non-zero values for these options (along with config_use_berg_velocity_solver = false), where the parallelism test would fail for differences only in uBergVelocity, vBergVelocity.
The interpolation call seaice_interpolate_cell_to_vertex is done in a block loop in the subroutine init_berg_velocity, so I believe this subroutine would need to be pulled out of that block loop.

Recording this as an issue for my own tracking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant