Michael Duda
Michael Duda
> @mgduda Could you review this one please? Certainly. Should this PR be targeting the `develop` branch or the `release-v4.4.2` branch?
At least initially, I think what we're after is "unsupported." I see now that the word *support* has very specific meaning in CESM. I'll update the Issue accordingly.
> In that case, I think we should have an MPAS test category we could use to make sure all the cases can run in principle. We don't have this...
As a temporary workaround, if testing without the frontogenesis gravity wave drag (?) scheme is acceptable, setting `use_gw_front = false` in CAM's namelist might suffice. It looks like the `cell_gradient_coef_x`...
@gold2718 This seems like something worth having. I suppose I was only vaguely aware of this Issue. Shall I assign myself?
@islas @weiwangncar Should we target the `release-v4.6.1` branch with this PR?
@islas Can you update the PR description to note that `main` returns an `int` rather than `void`?
Agreed that we shouldn't try to "correct" the computation, as we may be missing some important details. As a final comment, I'd suggest trying to match the whitespace convention used...
@HathewayWill I don't think using `mpiifort` is what we're after. My understanding is that `mpiifort` is the MPI Fortran compiler wrapper for Intel's MPI library, while I believe we're trying...
I think this is something worth considering for the next WPS release.