Improve definitions and usage of "surface" vs "surface_adjacent_layer" for wind variables, as well as "direction" - #155
Conversation
…rom_direction`, and update a few more missed variables containing "wind_at_surface"
|
@mkavulich Can you confirm that "wind_at_surface_adjacent_layer" is "wind at the lowest model layer" ? |
I see. It is. |
svahl991
left a comment
There was a problem hiding this comment.
Could we also look to see which wind *_at_surface_adjacent_layer names do not have a corresponding *_at_10m name, and add them where necessary? Then they're there if/when we need them.
FWIW, I've encountered someone else who finds the "surface adjacent layer" wording more confusing than "lowest model layer". Personally, I don't have a problem with either one, and think both are better than the "at surface" being replaced. I don't know if I consider it worth changing, but I thought I'd mention it. |
|
@svahl991 I'm unable to view the JCSDA internal repository, so let me know if there's any specific context I'm missing. The That said, I can also envision scenarios where "surface-adjacent layer" is not synonymous with "bottom layer of atmosphere". For example, an atmospheric state interpolated to pressure coordinates can easily have "layers" that are below the physical surface (I clearly remember this scenario giving me fits in my masters thesis work!). In that case, if we did decide to change the wording, something like "atmospheric layer immediately above the surface" would be required. I will bring this up for wider discussion at next week's meeting. |
|
To be clear, I realize changing the But here's the PR comment you couldn't follow the link to. It came in the context of a discussion of making the name changes that are in this PR within JEDI code. It was similar to @jianjunj above, so I just wanted to mention it.
|
Description
As discussed in issues #151 and #153, there are shortcomings in both the names and definitions surrounding near-surface wind. This PR implements the changes discussed in those issues:
wind_at_surfacevariables are nowwind_at_surface_adjacent_layerwind_to_directionin addition towind_from_directiondirectionis given a definition in the rulesIssues
Resolves #151, #153