<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:Consolas;
panose-1:2 11 6 9 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:#0563C1;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:#954F72;
text-decoration:underline;}
p.msonormal0, li.msonormal0, div.msonormal0
{mso-style-name:msonormal;
margin:0in;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
span.EmailStyle19
{mso-style-type:personal-reply;
font-family:"Calibri",sans-serif;
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style>
</head>
<body lang="EN-US" link="#0563C1" vlink="#954F72">
<div class="WordSection1">
<p class="MsoNormal">Dear Shengzhe,<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Thank you for identifying the bug in cal_section_trsp and telling us about it. Would you mind please sending the code with the bug fix to
<a href="mailto:ian.fenty@jpl.nasa.gov">ian.fenty@jpl.nasa.gov</a> and <a href="mailto:Ou.Wang@jpl.nasa.gov">
Ou.Wang@jpl.nasa.gov</a>. One of us will make sure that the code is fixed and that you are given credit!
<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Of course, if you are familiar with github, you can propose the code change yourself. Here is the python package repository:
<a href="https://github.com/ECCO-GROUP/ECCOv4-py">https://github.com/ECCO-GROUP/ECCOv4-py</a><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">-Ian<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:12.0pt;color:black">From: </span></b><span style="font-size:12.0pt;color:black"><ecco-support-bounces@mit.edu> on behalf of "Chen, Shengzhe" <schen26@albany.edu><br>
<b>Reply-To: </b>"ECCO support list, wider membership" <ecco-support@mit.edu><br>
<b>Date: </b>Tuesday, July 23, 2019 at 5:58 PM<br>
<b>To: </b>"ecco-support@mit.edu" <ecco-support@mit.edu><br>
<b>Subject: </b>[EXTERNAL] [ecco-support] Potential errors in ecco-v4-py python package<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div id="divtagdefaultwrapper">
<p><span style="font-size:12.0pt;color:black">Hi there,<o:p></o:p></span></p>
<p><span style="font-size:12.0pt;color:black">This is Shengzhe Chen, 3rd-year graduate student from UAlbany. Thanks to Dr. Ian Fenty, I am able to learn from the python packages to process ECCOv4 data. <o:p></o:p></span></p>
<p><span style="font-size:12.0pt;color:black"><o:p> </o:p></span></p>
<p><span style="font-size:12.0pt;color:black">Recently I was trying to use cal_section_trsp to calculate the salt and heat transport through some straits in the Arctic. I found that the results are very different from the observation data (e.g. mooring data)
in terms of magnitude. The observation data often show heat transport in the magnitude of TW, where the results generated by ECCOv4 show magnitude of hundreds of TW (PW to TW was done). I then went through the cal_section_trsp codes and compare them with gcmfaces
Matlab codes written by Dr. Gael Forget. I found that in heat transport and salt transport part of ECCO-v4-py, a Multiplication sign instead of a plus sign is assigned to combine the advective flux of potential temperature (e.g. </span><span style="font-size:9.0pt;font-family:Consolas;color:#032F62">ADVx_TH</span><span style="font-size:12.0pt;color:black">)
and diffusive flux of potential temperature (e.g. </span><span style="font-size:9.0pt;font-family:Consolas;color:#032F62">DFxE_TH</span><span style="font-size:12.0pt;color:black">). <o:p></o:p></span></p>
<p><span style="font-size:12.0pt;color:black">After I changed the sign from multiplication to plus, the results look more similar to observation data.<o:p></o:p></span></p>
<p><span style="font-size:12.0pt;color:black"><o:p> </o:p></span></p>
<p><span style="font-size:12.0pt;color:black">Thank you again for the great work of Python package which helps me a lot to take advantage of ECCOv4 data.<o:p></o:p></span></p>
<p><span style="font-size:12.0pt;color:black"><o:p> </o:p></span></p>
<p><span style="font-size:12.0pt;color:black">Best,<o:p></o:p></span></p>
<p><span style="font-size:12.0pt;color:black">Shengzhe<o:p></o:p></span></p>
</div>
</div>
</body>
</html>