-
Notifications
You must be signed in to change notification settings - Fork 13
Winter wheat changes by Theresa Boas #45
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
base: master
Are you sure you want to change the base?
Conversation
Patching strategy# *****************************
# 1. Set useful shell variables
# *****************************
# Directories
eCLM_DIR="/path/to/eCLM"
WORKDIR="/some/path/..."
# Winter wheat git info
WW_BRANCH="winter_wheat_cc"
WW_COMMIT_HASH="0b38504117e9d4b7e94586bdd26c631cc855266e"
# ************************************************************
# 2. Generate winter wheat patch file from Theresa's CLM5 fork
# ************************************************************
# Clone Theresa's winter wheat CLM5 branch
cd ${WORKDIR}
git clone -b ${WW_BRANCH} https://github.com/tboas/CTSM.git CLM5_WW
cd CLM5_WW
# Check changed files
git diff ${WW_COMMIT_HASH} --stat -- src
# Create patch file
git diff ${WW_COMMIT_HASH} -- src > ww_patch.diff
WW_PATCH=$(realpath ww_patch.diff)
# **************************************
# 3. Apply winter wheat patches to eCLM
# **************************************
# Go to eCLM and create winter wheat branch
cd ${eCLM_DIR}
git checkout -b ${WW_BRANCH}
# Transform CLM5 paths in patch file to equivalent paths in eCLM.
sed -i 's#/src/#/src/clm5/#g' ${WW_PATCH}
# Apply patch
git apply --ignore-space-change --ignore-whitespace ${WW_PATCH}
# Check if eCLM diffs correspond to original WW patch
git diff > ww_eclm_patch.diff
git diff --no-index ww_eclm_patch.diff ${WW_PATCH} # <- should have no diffs here! |
|
Do you have a comparison between clm5-Winter_Wheat and eclm-Winter_Wheat? |
|
Update 24-Jan-2025: This PR is on hold until these conditions are met:
|
|
@kvrigor could you have a look at this? |
|
@AGonzalezNicolas this can only be merged after #54 and manual verification. This PR changes the default BGP/BGC behavior of eCLM so we have to be extra careful. |
|
After the SDLTS meeting today, it was decided to merge Olga's Fruit Tree implementation (#94) first before this PR. @AGonzalezNicolas Do you know if this reference winterwheat test case still valid? https://github.com/HPSCTerrSys/REFcase_clm5.0_FTWC_for_WC ? This looks like what we exactly need to validate this PR. |
Equivalent to winter wheat CLM5 patches by @tboas. Modifications include:
For additional information and results please see: