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

test script for acme_land_developer fail on edison #1336

Closed
jinyuntang opened this issue Mar 22, 2017 · 4 comments
Closed

test script for acme_land_developer fail on edison #1336

jinyuntang opened this issue Mar 22, 2017 · 4 comments
Assignees

Comments

@jinyuntang
Copy link
Contributor

The script for doing test acme_land_developer now fails on edison. It was fine last month before the new cime. Now it no longer works for me. I describe my experience in detail below.

With the following c-shell script, I successfully generated baseline.

#!/bin/csh -f

setenv MY_BASELINE_DIR /project/projectdirs/acme/jinyun/acme_baselines/

setenv PROJECT acme
setenv BASELINE_NAME 7a8187b
setenv TEST_ID ${BASELINE_NAME}

./create_test
acme_land_developer
--baseline-root ${MY_BASELINE_DIR}
-p ${PROJECT}
-b ${BASELINE_NAME}
-t ${TEST_ID}
-g
-v

However, when it failed when I use the following script to generate the case for comparison
#!/bin/csh -f

setenv MY_BASELINE_DIR /project/projectdirs/acme/jinyun/acme_baselines/

setenv PROJECT acme
setenv BASELINE_NAME 7a8187b
setenv TEST_ID git log -n 1 --format=%h

./create_test
acme_land_developer
--baseline-root ${MY_BASELINE_DIR}
-p ${PROJECT}
-b ${BASELINE_NAME}
-t ${TEST_ID}
-g
-v

The error message is as follows.

ERROR: Baseline directories already exists ['/project/projectdirs/acme/jinyun/acme_baselines/intel/7a8187b/ERS.f09_g16.I1850CLM45CN.edison_intel', '/project/projectdirs/acme/jinyun/acme_baselines/intel/7a8187b/ERS.f09_g16.IMCLM45BC.edison_intel', '/project/projectdirs/acme/jinyun/acme_baselines/intel/7a8187b/ERS.f19_f19.I1850CLM45CN.edison_intel', '/project/projectdirs/acme/jinyun/acme_baselines/intel/7a8187b/ERS.f19_f19.IM1850CLM45CN.edison_intel', '/project/projectdirs/acme/jinyun/acme_baselines/intel/7a8187b/ERS.f19_f19.IMCLM45.edison_intel', '/project/projectdirs/acme/jinyun/acme_baselines/intel/7a8187b/ERS.ne11_oQU240.I20TRCLM45.edison_intel', '/project/projectdirs/acme/jinyun/acme_baselines/intel/7a8187b/SMS.hcru_hcru.I1850CRUCLM45CN.edison_intel', '/project/projectdirs/acme/jinyun/acme_baselines/intel/7a8187b/SMS_Ly3_P1x1.1x1_smallvilleIA.ICLM45CNCROP.edison_intel.force_netcdf_pio']

@jgfouca
Copy link
Member

jgfouca commented Mar 22, 2017

Just add -o to your flags to create_test.

@jgfouca jgfouca self-assigned this Mar 22, 2017
@jinyuntang
Copy link
Contributor Author

@jgfouca, so this is on purpose to overwrite the baseline directories?

@jgfouca
Copy link
Member

jgfouca commented Mar 22, 2017

Yes, the old behavior was to silently blow away any previous baselines. The new behavior requires the user to specify -o.

@jinyuntang
Copy link
Contributor Author

@jgfouca , Thanks, then I'll close the issue then.

agsalin pushed a commit that referenced this issue Apr 13, 2017
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

No branches or pull requests

2 participants