<div dir="ltr"><div><br></div><div>GENERAL NOTES:<br><br></div><div>Having a mix of three scripting languages may annoy NCO.  It is not realistic to change that now.  Just be aware they may be annoyed.<br></div><div><br>J-JOB ISSUES:<br><br></div>The dates and times should not be hard-coded.  <br><div>export PDY=20160928<br>priorymdh=2016092812<br>cyc=18                                                                           <br><br></div><div>NWROOT should not be hard-coded.<br>NWROOT=/gpfs/hps/emc/hwrf/noscrub/Lin.L.Zhu/save<br><br></div><div>EX-SCRIPT ISSUES:<br><br></div><div>exhmon_cpl - make_symlinks does not pass a logger<br><br></div><div>exhmon_hycominit*.py - these three are identical except for one string.  You should have one script, and pass the string as an argument or environment variable.<br><br></div><div>exhmon_post*.py - same thing: three nearly identical scripts<br></div><div><br>exhmon_nps_bc.py - many calls lack a logger= argument<br><br>exhmon_nps_ic.py - a few calls lack logger= arguments<br><br></div><div>USH ISSUES:<br><br></div><div>conf.py - isn&#39;t this just a rewrite of the python core ConfigParser module?<br><br></div><div>hmon_generate_vitals.py - having dev paths in this script may annoy NCO<br><br></div><div>Sincerely,<br></div><div>Sam Trahan<br></div></div>