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.

Sign up for GitHub

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

Jump to bottom

It looks like we no longer support horiz_winds as a nudging variable #2747

Open
AaronDonahue opened this issue Mar 7, 2024 · 1 comment
Open

It looks like we no longer support horiz_winds as a nudging variable #2747

AaronDonahue opened this issue Mar 7, 2024 · 1 comment
Labels
bug Something isn't working nudging pertains to the nudging process

Comments

@AaronDonahue
Copy link
Contributor

With the changes we made to support U and V as nudging fields it looks like we've lost support for horiz_winds as a nudging field. This is an issue for cases where the data used for nudging comes from older SCREAM runs and we don't have U and V as output, instead they are combined into horiz_winds.
@AaronDonahue AaronDonahue added bug Something isn't working nudging pertains to the nudging process labels Mar 7, 2024
@bartgol
Copy link
Contributor

bartgol commented Mar 7, 2024

I think this was on purpose. We are assuming all nudged vars have layout (ncols,nlevs). horiz_winds does not fit into this pattern.

How important is it to use old scream runs as nudging data?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working nudging pertains to the nudging process
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bartgol @AaronDonahue