Skip to content

ufs_public_release: remove libxml2#46

Merged
DusanJovic-NOAA merged 5 commits into
ufs-community:ufs_public_releasefrom
climbfuji:remove_libxml2
Feb 6, 2020
Merged

ufs_public_release: remove libxml2#46
DusanJovic-NOAA merged 5 commits into
ufs-community:ufs_public_releasefrom
climbfuji:remove_libxml2

Conversation

@climbfuji
Copy link
Copy Markdown
Collaborator

@climbfuji climbfuji commented Feb 6, 2020

Remove reference to libxml2, this not required for the CCPP static build. Avoids that users need to install libxml2 on their systems for something that doesn't get used at all.

Also fixes a bug for compiling on gaea (rename cmake/configure_gaea.cmake to cmake/configure_gaea.intel.cmake). Then, it is possible to compile the code using either

cd tests
./compile_cmake.sh $PWD/.. gaea.intel '32BIT=Y CCPP=Y STATIC=Y SUITES=FV3_GFS_v15p2,FV3_GFS_v16beta' 2>&1 | tee compile.log

or

module use -a $PWD/modulefiles/gaea.intel
module load fv3
export NETCDF=$NETCDF_DIR
./build.sh 2>&1 | tee build.log

However, for users outside of the ESRL organization, this still requires GFDL helpdesk to fix the permissions (cannot read /lustre/f2/pdata/esrl, permissions underneath are correct). I opened a ticket for this and they are working on it.

Associated PRs:

NCAR/ccpp-framework#261
NOAA-EMC/ufsatm#55
#46
Also: NOAA-EMC/ufsatm#56

For regression testing, see below.

@climbfuji
Copy link
Copy Markdown
Collaborator Author

Regression testing on hera.intel, cheyenne.intel and cheyenne.gnu: all tests pass. Regression test logs have been updated in this PR.

rt_cheyenne_gnu.log
rt_cheyenne_intel.log
rt_hera_intel.log

@climbfuji climbfuji marked this pull request as ready for review February 6, 2020 12:58
@climbfuji
Copy link
Copy Markdown
Collaborator Author

Ready to go!

@DusanJovic-NOAA DusanJovic-NOAA merged commit ff711b8 into ufs-community:ufs_public_release Feb 6, 2020
XiaSun-Atmos pushed a commit to XiaSun-Atmos/ufs-weather-model that referenced this pull request Aug 8, 2020
climbfuji pushed a commit to climbfuji/ufs-weather-model that referenced this pull request Nov 21, 2020
…_from_develop_20201118

Update gsd/develop from develop 2020/11/18
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.

3 participants