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

ufs_public_release: remove libxml2 #46

Merged

Conversation

climbfuji
Copy link
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/fv3atm#55
#46
Also: NOAA-EMC/fv3atm#56

For regression testing, see below.

@climbfuji
Copy link
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
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