Skip to content

Upgrade preview_run.#2539

Merged
jgfouca merged 3 commits intomasterfrom
jgfouca/upgrade_preview_run
May 4, 2018
Merged

Upgrade preview_run.#2539
jgfouca merged 3 commits intomasterfrom
jgfouca/upgrade_preview_run

Conversation

@jgfouca
Copy link
Contributor

@jgfouca jgfouca commented May 3, 2018

  1. Redo the formatting of output to make it prettier
  2. Add some output for key derived attributes
  3. Add logger.info suggestion to user to run preview_run

Test suite: by-hand, code-checker
Test baseline:
Test namelist changes:
Test status: bit for bit

Fixes [CIME Github issue #]

User interface changes?: preview_run output improvements

Update gh-pages html (Y/N)?: N

Code review: @jedwards4b

1) Redo the formatting of output to make it prettier
2) Add some output for key derived attributes
3) Add logger.info suggestion to user to run preview_run
@jgfouca
Copy link
Contributor Author

jgfouca commented May 3, 2018

Example of new output:

CASE INFO:
  nodes: 1
  total tasks: 24
  tasks per node: 24
  thread count: 2

BATCH INFO:
  FOR JOB: case.test
    ENV:
      module command is /usr/bin/modulecmd python purge 
      module command is /usr/bin/modulecmd python load sems-env acme-env sems-git sems-python/2.7.9 sems-cmake/2.8.12 sems-gcc/5.3.0 sems-openmpi/1.8.7 sems-netcdf/4.4.1/exo_parallel
      Setting Environment NETCDFROOT=/projects/sems/install/rhel6-x86_64/sems/tpl/netcdf/4.4.1/gcc/5.3.0/openmpi/1.8.7/exo_parallel
      Setting Environment OMP_STACKSIZE=64M
      Setting Environment OMP_PROC_BIND=spread
      Setting Environment OMP_PLACES=threads
      Setting Environment PNETCDFROOT=/projects/sems/install/rhel6-x86_64/sems/tpl/netcdf/4.4.1/gcc/5.3.0/openmpi/1.8.7/exo_parallel
    SUBMIT CMD:
      case.test

MPIRUN:
  mpirun  -np 24  --map-by ppr:12:socket:PE=2 --bind-to hwthread /home/jgfouca/acme/scratch/SMS_P24x2.f19_g16_rx1.A.melvin_gnu.20180503_131747_xuv3a7/bld/e3sm.exe  >> e3sm.log.$LID 2>&1 

@jedwards4b
Copy link
Contributor

@jgfouca failing travis-ci testing, please address.

@jgfouca
Copy link
Contributor Author

jgfouca commented May 4, 2018

@jedwards4b , I cannot reproduce this error on my machine

@jgfouca
Copy link
Contributor Author

jgfouca commented May 4, 2018

I think it has something to do with Travis using newer versions than I have.

@jgfouca
Copy link
Contributor Author

jgfouca commented May 4, 2018

OK, that fixed it.

@jgfouca jgfouca merged commit 69ddf88 into master May 4, 2018
@jgfouca jgfouca deleted the jgfouca/upgrade_preview_run branch May 4, 2018 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants