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

GitHub Issue NOAA-EMC/GSI#17. Bug fixes for regional EnKF, GNU compilers; change build.comgsi; move gsdcloud to src. #18

Merged
merged 1 commit into from
Jun 17, 2020

Conversation

guoqing-noaa
Copy link
Contributor

  1. added FV3GFS_NCIO_Fortran_FLAGS for GNU compilers

  2. Moved gsdcloud to src/, make appropriate changes in CMakeLists.txt and a few bug fixes

  3. pietc.f90 and pvqc_tables.f90

    use kinds, only: dp,dpc
--> use kinds, only: r_kind, dp, dpc

This change is to avoid compiling error

error #6580: Name in only-list does not exist or is not accessible. [DP]

Don't know the exact reason, but either (1) removing all #ifdef block in kinds.F90 or (2) use r_kind first can solve this problem.

  1. updated util/EnKF/arw/src/enspreproc_regional.fd/CMakeLists.txt

  2. bug fixes for GNU compilers and regional EnKF.

  3. updated ush/build.comgsi for community users

change build.comgsi
move GSD/gsdcloud to src/
@hu5970
Copy link
Collaborator

hu5970 commented Jun 15, 2020

Should we change directory name from GSD to NoVarCloudAnal for clarification.

@guoqing-noaa
Copy link
Contributor Author

Should we change directory name from GSD to NoVarCloudAnal for clarification.

I would say better to leave it as is. I had thought to change GSD to GSL. But keeping it as is will involve less changes and many community users are already familiar with this GSD naming convection. May change it in the future if warranted. Thanks!

@MichaelLueken MichaelLueken changed the title bug fixes for regional EnKF, GNU compilers; change build.comgsi; move gsdcloud to src/ GitHub Issue NOAA-EMC/GSI#17. Bug fixes for regional EnKF, GNU compilers; change build.comgsi; move gsdcloud to src. Jun 17, 2020
@MichaelLueken MichaelLueken merged commit fcec2a9 into NOAA-EMC:master Jun 17, 2020
shoyokota pushed a commit to shoyokota/GSI that referenced this pull request Jul 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug fixes for regional EnKF, GNU compilers; change build.comgsi; move gsdcloud to src/
4 participants