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

Hotfix/no_tiles_in process #316

Merged
merged 3 commits into from
Sep 30, 2020
Merged

Conversation

weiyuan-jiang
Copy link
Contributor

  1. fix the situation when there is no tiles in a process
  2. Determine NUM_LDAS_ENSEMBLE entry in lenkf.j from LDAS.rc, replacing the independent, ldas_setup-generated specification in lenkf.j
  3. use the relative link path. When the whole directory is moved (like CURRENT to BASELINE) the link would be still valid.

2) Determine NUM_LDAS_ENSEMBLE entry in lenkf.j from LDAS.rc, replacing the independent, ldas_setup-generated specification in lenkf.j
@gmao-rreichle
Copy link
Contributor

gmao-rreichle commented Sep 25, 2020

@weiyuan-jiang : I (re-)fixed the use of the optional argument "obsbias_ok_tmp" and added a few comments, see commit 29deeed
Did you already test this branch? (My commit should not change the test result.)

cc: @biljanaorescanin

@weiyuan-jiang
Copy link
Contributor Author

good catch. I have test this branch and it seems zero-diff

Copy link
Contributor

@gmao-rreichle gmao-rreichle left a comment

Choose a reason for hiding this comment

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

Approved. I understand the comment by @weiyuan-jiang to mean that the branch is 0-diff vs develop in the current test suite. What remains to be seen is if the branch is 0-diff between the (old) 1d decomposition (w/ optimization) and the new 2d decomposition functionality.
In any case, I'm ok with merging the branch at this time because it is 0-diff in the current test suite and because it fixes other things.
But we may need additional modifications to achieve 0-diff between the 1d and 2d decompositions.
cc: @biljanaorescanin

@biljanaorescanin biljanaorescanin merged commit 455b31e into develop Sep 30, 2020
@biljanaorescanin biljanaorescanin deleted the hotfix/no_tiles_in_process branch September 30, 2020 14:25
biljanaorescanin added a commit that referenced this pull request Oct 13, 2020
@gmao-rreichle gmao-rreichle linked an issue Dec 16, 2020 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

optimize processor layout (2d)
3 participants