pbs update/fix for cheyenne

[jmccreight]

🤦 🤕 but better docs!

• at least for impi (our standard mpi implementation) the parsing of the node file entries to only take the actual node name and not complete identifier (e.g. r8i3n16 and not r8i3n16.ib0.cheyenne.ucar.edu) seemed to stop working with module/impi updates. so now the full identifier is retained. The currently working mpi command syntax on cheyenne is: 'mpi_cmd': '"mpiexec -hosts {nodelist} -n {nproc} ./wrf_hydro.exe"'
• in the above you note that i changed {hostname} to {nodelist} to better reflect what is actually substituted in the mpi command. it’s a list of nodes that are comma separated, not a single name
• added parallel_teams_run to the documentation

[pep8speaks]

Hello @jmccreight! Thanks for updating this PR.

• In the file wrfhydropy/core/ioutils.py:

Line 455:1: E302 expected 2 blank lines, found 1

• In the file wrfhydropy/core/teams.py:

Line 246:13: E129 visually indented line with same indent as next logical line Line 246:18: E201 whitespace after '[' Line 247:17: E117 over-indented

• In the file wrfhydropy/tests/test_cycle.py:

Line 1228:82: E202 whitespace before ')' Line 1259:18: E131 continuation line unaligned for hanging indent Line 1259:101: E501 line too long (117 > 100 characters) Line 1261:18: E131 continuation line unaligned for hanging indent Line 1261:101: E501 line too long (117 > 100 characters) Line 1263:18: E131 continuation line unaligned for hanging indent Line 1263:101: E501 line too long (117 > 100 characters) Line 1387:14: E131 continuation line unaligned for hanging indent Line 1387:101: E501 line too long (113 > 100 characters) Line 1390:14: E131 continuation line unaligned for hanging indent Line 1390:101: E501 line too long (113 > 100 characters) Line 1393:14: E131 continuation line unaligned for hanging indent Line 1393:101: E501 line too long (113 > 100 characters)

• In the file wrfhydropy/tests/test_ensemble.py:

Line 408:96: E202 whitespace before '}' Line 434:1: W293 blank line contains whitespace Line 500:101: E501 line too long (113 > 100 characters) Line 502:101: E501 line too long (113 > 100 characters) Line 504:101: E501 line too long (113 > 100 characters) Line 508:40: W291 trailing whitespace

• In the file wrfhydropy/tests/test_ioutils.py:

Line 62:42: E231 missing whitespace after ',' Line 154:1: W293 blank line contains whitespace

Comment last updated at 2020-09-02 16:54:02 UTC

[arezoorn]

I did try it for the sensitivity analysis across 6 nodes and it scaled as it should. So the changes are working fine!