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

Fixes for GNU compilation issues (from @XiaqiongZhou-NOAA) #22

Merged
merged 4 commits into from
Jun 22, 2020

Conversation

climbfuji
Copy link

@climbfuji climbfuji commented Jun 19, 2020

Fixes for GNU compilation issues: (NOAA-GFDL#32)

  • Remove trailing whitespace and any tabs
  • Add default values for nest_*offsets in fv_control
  • fix GNU out-of-bound error in driver/fvGFS/atmosphere.F90
  • Correct Z dimensions for driver/fvGFS/atmosphere.F90

All changes from @XiaqiongZhou-NOAA

Associated PRs:
https://github.com/ufs-community/ufs-weather-model/pull/151/files
NOAA-EMC/fv3atm#131
#22

For regression testing information, see below https://github.com/ufs-community/ufs-weather-model/pull/151/files.

@junwang-noaa
Copy link
Collaborator

Do we have the code updates in tools/fv_grid_tools.F90 for item1:

@XiaqiongZhou-NOAA
Copy link
Collaborator

This fix is removed in order not to change the baseline.

@climbfuji
Copy link
Author

Do we have the code updates in tools/fv_grid_tools.F90 for item1:

actually not, it was removed - I'll update the description.

@DusanJovic-NOAA DusanJovic-NOAA merged commit 205cc76 into NOAA-EMC:dev/emc Jun 22, 2020
tsupinie pushed a commit to tsupinie/GFDL_atmos_cubed_sphere that referenced this pull request Jul 20, 2020
* Fixes for GNU compilation issues (NOAA-GFDL#32)
* Fixes for mesh generation in init_grid (NOAA-GFDL#39)
* Remove trailing whitespace and any tabs
* Add default values for nest_*offsets in fv_control
* Remove the fix for mesh generation and it will be fixed later
* fix GNU out-of-bound error in driver/fvGFS/atmosphere.F90
* Correct Z dimensions in driver/fvGFS/atmosphere.F90

Co-authored-by: Xiaqiong Zhou <[email protected]>
XiaqiongZhou-NOAA pushed a commit to XiaqiongZhou-NOAA/GFDL_atmos_cubed_sphere that referenced this pull request Feb 26, 2021
* remove trailing whitespace and any tabs

* update the RELEASE.md with the FV3 technical memorandum

* semantic fix in RELEASE.md

* adds default values for nest_*offsets in fv_control

breaks up a too long line in fv_nesting.F90

* change default value of nestupdate to 7
@XiaqiongZhou-NOAA XiaqiongZhou-NOAA deleted the emc_gnu branch March 29, 2023 16:45
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.

5 participants