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

Fix examples.namelist namelist record name: domains #1340

Merged
merged 5 commits into from
Jan 7, 2021

Conversation

davegill
Copy link
Contributor

@davegill davegill commented Dec 16, 2020

TYPE: text only

KEYWORDS: namelist, readme

SOURCE: internal

DESCRIPTION OF CHANGES:
The trajectory examples.namelist was no longer providing sufficient info to run trajectories.

  1. One of the helpful suggestions had a misspelled namelist record. It should have been domains, but it was referred
    to as domain.
  2. There was no info on the additional options required: dm_has_traj.
  3. There was no mention of the now mandatory additional text file wrfinput_traj_d01 that has two namelist records.

LIST OF MODIFIED FILES:
M test/em_real/examples.namelist

TESTS CONDUCTED:

  1. This readme is not part of the build process, or part of compilable code. It will
    have no impact of results. Users likely also just skip past the incorrect namelist record
    identifier and grab the intended variable (num_traj). All in all, pretty small potatoes.
  2. The jenksins tests pass, but no surprise - since we did not change anything that is part of the build.

TYPE: text only

KEYWORDS: namelist, readme

SOURCE: internal

DESCRIPTION OF CHANGES:
In a readme file, one of the helpful suggestions had a misspelled namelist record.
It should be `domains`, and it was referred to as `domain`.

LIST OF MODIFIED FILES:
M	test/em_real/examples.namelist

TESTS CONDUCTED:
1. This readme is not part of the build process, or part of compilable code. It will
have no impact of results. Users likely also just skip past the incorrect namelist record
identifier and grab the intended variable (traj_opt). All in all, pretty small potatoes.
@davegill
Copy link
Contributor Author

@smileMchen
MIng,
This is a simple change in a readme. Would you review it please?

@davegill
Copy link
Contributor Author

davegill commented Jan 7, 2021

@smileMchen
Ming,
This is a small change to the examples.namelist. Would you review please. This does not change the WRF source code.

@smileMchen
Copy link
Collaborator

@davegill
Dave,
I will approve this PR. Thanks.

@davegill davegill merged commit 432f33d into wrf-model:release-v4.2.2 Jan 7, 2021
vlakshmanan-scala pushed a commit to scala-computing/WRF that referenced this pull request Apr 4, 2024
TYPE: text only

KEYWORDS: namelist, readme

SOURCE: internal

DESCRIPTION OF CHANGES:
The trajectory examples.namelist was no longer providing sufficient info to run trajectories.
1. One of the helpful suggestions had a misspelled namelist record. It should have  been `domains`, but it was referred 
to as `domain`.
2. There was no info on the additional options required: `dm_has_traj`.
3. There was no mention of the now mandatory additional text file `wrfinput_traj_d01` that has two namelist records.

LIST OF MODIFIED FILES:
M	test/em_real/examples.namelist

TESTS CONDUCTED:
1. This readme is not part of the build process, or part of compilable code. It will
have no impact of results. Users likely also just skip past the incorrect namelist record
identifier and grab the intended variable (num_traj). All in all, pretty small potatoes.
2. The jenksins tests pass, but no surprise - since we did not change anything that is part of the build.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants