-
Notifications
You must be signed in to change notification settings - Fork 17
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
Reconsideration of Add four wind CF variables #73? #77
Comments
I agree there is an unsatisfying inconsistency here, particularly for the |
For a shorter name, maybe we could use
|
@ncrossette - I do like the names that you propose. |
Is |
@climbfuji - I think |
Following the rules is confusing for a couple reasons.
Following the rules for constructing names: The names would be: |
What is the guideline for putting the medium as a suffix vs prefix, like it's done with these variables?:
|
Fair point. |
At least regarding the suffix/prefix situation regarding |
@dustinswales I've been thinking about the And I would make a similar argument for temperature and pressure: i.e. its the temperature OF the air rather than the more vague temperature in air (which leaves me asking 'temperature of what in air?'). And using the |
This does make sense to me, @ncrossette. For example, the standard name |
I agree prefix for medium, suffix for location. Locations should be
specific levels for example.
…On Mon, Oct 21, 2024 at 10:37 AM Nate Crossette ***@***.***> wrote:
@dustinswales <https://github.com/dustinswales> I've been thinking about
the air_... vs the ..._in_air prefix/suffixes, and for this set of
variables, I think the air_... prefix makes more sense semantically
because, physically, it is the divergence (or vorticity, etc.) *OF* the
air itself. Saying divergence _in_air sounds a little less precise since
the suffix doesn't make it explicitly sound like its the 'divergence OF the
air'.
And I would make a similar argument for temperature and pressure: i.e. its
the temperature OF the air rather than the more vague temperature in air
(which leaves me asking 'temperature of what in air?').
And using the air_ prefix would keep consistency with the temperature and
pressure variables.
—
Reply to this email directly, view it on GitHub
<#77 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AEIZ77GGIN6EOKEZ3RWUOEDZ4UUVXAVCNFSM6AAAAABPUJCMP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRXGE3TMMZZGE>
.
You are receiving this because you were assigned.Message ID:
***@***.***>
|
I also support @ncrossette's idea of using a prefix when referencing properties of the medium itself. Also, in terms of the original question, I think I am happy with the names suggested by @ncrossette above, with one change made to the vorticity to make it consistent with the others:
Of course I'm happy to hear if people disagree. |
I'm indifferent to whether we change from (in suffix): |
@MarekWlasak - in #78 these names are being updated which it appears you agree with. Just wanted to check about the variable: |
@ss421 - I think we can leave the 3D divergence variable for now (possibly called `divergence_of_wind_and_upward_air_velocity) as we are not using it at the moment. What is great is that now that the new variables are very clear. |
So I support
|
Following up on the discussion in #77, this updates the streamfunction, velocity_potential, vorticity, and divergence names to: air_horizontal_streamfunction air_horizontal_velocity_potential air_upward_absolute_vorticity air_horizontal_divergence Also, the description of air_upward_absolute_vorticity is updated to indicate that it is the kth/upward component of the curl.
In #73 we came up with 4 new wind CF variable names: (I approved this before the final choice for the names was chosen!)
I apologize to all concerned who have been implementing the new variables and the work they have done. (In particular @svahl991 @shlyaeva @ncrossette)
The new variables are:
atmosphere_horizontal_streamfunction
: Scalar function describing the stream lines of the windatmosphere_horizontal_velocity_potential
: Scalar potential of the windatmosphere_upward_absolute_vorticity
: The kth component of the curl of the vector wind field. The curl itself is assuming wind is a 3D object.divergence_of_wind
: The (horizontal) divergence of the 2-D vector wind fieldI have a few concerns above these names:
divergence_of_the_wind
(although consistent with the CF understanding that a wind is by definition 2D) is misleading to most model developers who consider the wind as a 3D vector. This becomes problematic in high resolution modelling especially around mountains. It is only a matter of time where someone will naively misuse this name and consider it to mean the 3D divergence. This is why adding "horizontal_" in front would greatly reduce the risk of misuse of the variable.divergence_of_wind
not to be misused.Suggestions
atmosphere_horizontal_streamfunction
->horizontal_streamfunction
atmosphere_horizontal_velocity_potential
->horizontal_velocity_potential
atmosphere_upward_absolute_vorticity
->upward_absolute_vorticity
We need to update the description to include The kth component of the curl of the vector wind field.divergence_of_wind
(keep as is)divergence_of_wind_and_upward_air_velocity
(add this to help clarity - 3D divergence!)The text was updated successfully, but these errors were encountered: