<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Hi Hong<div class=""><br class=""></div><div class="">Thanks for taking a look. I am bit confused about the suggested modifications but maybe they aren’t actually needed and I don't want to introduce further delay.&nbsp;</div><div class=""><br class=""></div><div class="">Why not just (1) make sure that `nctiles_grid/` in r3 and r4 is the same as in r2 to avoid disrupting returning users, (2) move the other `GRID*` stuff to a separate folder to avoid confusing new users, and (3) document in your READMEs the circumstances when each grid folder should be used?&nbsp;</div><div class=""><br class=""></div><div class="">Ultimately it’s obviously up you guys at JPL to decide on what you want to do to resolve this situation. I am just worried we might be losing more users&nbsp;<font color="#000000" class=""><span style="caret-color: rgb(255, 255, 255);" class="">until&nbsp;JPL remedies the confusion that’s emerged from the reformatting.&nbsp;</span></font><span style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0);" class="">The added documentation in READMEs is probably needed regardless of anything else.</span></div><div class=""><span style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0);" class=""><br class=""></span></div><div class=""><span style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0);" class="">Hope this helps.&nbsp;</span></div><div class=""><span style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0);" class=""><br class=""></span></div><div class=""><font color="#000000" class=""><span style="caret-color: rgb(0, 0, 0);" class="">Best,</span></font></div><div class=""><font color="#000000" class=""><span style="caret-color: rgb(0, 0, 0);" class="">Gael</span></font></div><div class=""><br class=""></div><div class=""><br class=""></div><div class=""><br class=""></div><div class=""><br class=""></div><div class=""><div><blockquote type="cite" class=""><div class="">On May 3, 2020, at 7:44 PM, Zhang, Hong (US 398K) &lt;<a href="mailto:hong.zhang@jpl.nasa.gov" class="">hong.zhang@jpl.nasa.gov</a>&gt; wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">Hi Gael,<br class="">I took a look. We need to modify<br class="">grid_load.m and read_nctiles.m for r4.<br class=""><br class="">"read_nctiles.m" needs two minor fixes<br class="">1) <a href="https://github.com/MITgcm/gcmfaces/blob/master/gcmfaces_IO/read_nctiles.m#L64" class="">https://github.com/MITgcm/gcmfaces/blob/master/gcmfaces_IO/read_nctiles.m#L64</a><br class="">basedir = './'; ==&gt; basedir = '.';<br class="">2) <a href="https://github.com/MITgcm/gcmfaces/blob/master/gcmfaces_IO/read_nctiles.m#L190" class="">https://github.com/MITgcm/gcmfaces/blob/master/gcmfaces_IO/read_nctiles.m#L190</a><br class=""> &nbsp;&nbsp;&nbsp;t0=tt(1)-1; ==&gt; &nbsp;&nbsp;&nbsp;&nbsp;t0=tt(1);<br class=""><br class="">It's hard to reconcile "GRID*.nc" from r3 and "ECCO-GRID*.nc" from r4<br class="">in "grid_load.m" because they have different field names.<br class="">One work-around is to copy <br class=""><a href="https://ecco.jpl.nasa.gov/drive/files/Version4/Release3/nctiles_grid/GRID.nc" class="">https://ecco.jpl.nasa.gov/drive/files/Version4/Release3/nctiles_grid/GRID.nc</a><br class="">to<br class="">https://ecco.jpl.nasa.gov/drive/files/Version4/Release4/nctiles_grid/<br class=""><br class="">I did a test as above and it's OK.<br class="">But it took ~204 seconds to load 312 monthly v4 ETAN from 312 files<br class="">as opposed to 1.4 seconds to load 288 monthly v3 ETAN from 13 files.<br class="">Perhaps this arrangement has more benefits to ECCO-v4.py<br class=""><br class="">cheers<br class="">Hong<br class="">_______________________________________________<br class="">ecco-support mailing list<br class="">ecco-support@mit.edu<br class="">http://mailman.mit.edu/mailman/listinfo/ecco-support<br class=""></div></div></blockquote></div><br class=""></div></body></html>