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

Update the readme file with a note about the 'generic' configuration #175

Merged
merged 3 commits into from
May 12, 2022
Merged

Update the readme file with a note about the 'generic' configuration #175

merged 3 commits into from
May 12, 2022

Conversation

gmao-msienkie
Copy link
Contributor

Updated the readme_init_coeffs file with a note about how it uses the 'gsi.rc.tmpl' to get 'dtype', 'dplat' and 'dsis' for each satellite.

@gmao-msienkie gmao-msienkie added documentation Improvements or additions to documentation 0 diff trivial The changes in this pull request are trivially zero-diff (documentation, build failure, &c.) labels May 3, 2022
@gmao-msienkie gmao-msienkie requested a review from a team as a code owner May 3, 2022 17:53
gmao-msienkie and others added 2 commits May 5, 2022 17:19
@gmao-msienkie
Update readme_init_coeffs.txt
adfdff1 
Edited the file to remove some information that had been included twice.
@gmao-msienkie
Adding a modification to init_coeffs.f90 for the OSSE geoimr
8215c94 
instruments - add to mean_only list (no crosstrack estimate)
@gmao-msienkie gmao-msienkie added 0 diff The changes in this pull request have verified to be zero-diff with the target branch. and removed 0 diff trivial The changes in this pull request are trivially zero-diff (documentation, build failure, &c.) labels May 6, 2022
@gmao-msienkie
Copy link
Contributor Author

Whoops - now adding a code change as well. Results should be zero diff - except for geoimr which is not used in the standard build.

rtodling
rtodling approved these changes May 12, 2022
View reviewed changes
Copy link
Collaborator

@rtodling rtodling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Meta.

@rtodling rtodling merged commit c115b54 into develop May 12, 2022
@gmao-msienkie gmao-msienkie deleted the feature/msienkie/init_coeffs_readme branch January 31, 2023 17:20
@gmao-msienkie gmao-msienkie restored the feature/msienkie/init_coeffs_readme branch January 31, 2023 17:21
@gmao-msienkie gmao-msienkie deleted the feature/msienkie/init_coeffs_readme branch January 31, 2023 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rtodling rtodling rtodling approved these changes

Assignees
No one assigned
Labels
0 diff The changes in this pull request have verified to be zero-diff with the target branch. documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gmao-msienkie @rtodling