<div dir="ltr">Hi J-M<div><br></div><div>Ah ok I remember now. Well, not completely, but I think it was for similar reasons to the OpenAD_* calls in initialise_fixed.F.</div><div><br></div><div>this sounds great then, if the s/r name could be moved to the list in the code_oad file. (the idea is that someone should do as little digging as needed to create an openad setup of streamice, but having the template in the verification folder works...)</div><div><br></div><div>Cheers!<br></div><div>Dan</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 18, 2018 at 11:37 PM, Jean-Michel Campin <span dir="ltr"><<a href="mailto:jmc@mit.edu" target="_blank">jmc@mit.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Dan,<br>
<br>
OK, I missed a "special" OpenAD feature in packages_init_fixed.F (lines 471-475)<br>
that justifies to have streamice_init_fixed.f in ad_diff.list<br>
<br>
We could just leave with this (TAF complains but does not seem very serious ?)<br>
or we could move "streamice_init_fixed.f" to the local: code_oad/code_ad_diff.list<br>
There is already a file like this in verification/halfpipe_<wbr>streamice/code_oad<br>
so it would not involve too many changes (and would be cleaner for TAF build).<br>
<br>
Let me know what you prefer and will do it soon after.<br>
<br>
Cheers,<br>
Jean-Michel<br>
<br>
On Thu, Jan 18, 2018 at 08:13:09AM +0000, Daniel Goldberg wrote:<br>
> Hi J-M<br>
><br>
> I???m not sure as this might be borne out of improper understanding of the ad_diff.list file or it could have to do with openad which makes use of the same file I believe. I will check but I???m pretty sure it can be removed.<br>
<div class="HOEnZb"><div class="h5">><br>
> Dan<br>
><br>
> Sent from my iPhone. Apologies for brevity.<br>
><br>
> > On Jan 17, 2018, at 11:42 PM, Jean-Michel Campin <<a href="mailto:jmc@mit.edu">jmc@mit.edu</a>> wrote:<br>
> ><br>
> > Hi Dan,<br>
> ><br>
> > Do you remember why you added "streamice_init_fixed.f" in<br>
> > streamice_ad_diff.list (CVS revision 1.8, on Mar 07, 2015) ?<br>
> ><br>
> > S/R STREAMICE_INIT_FIXED is called from packages_init_fixed.F<br>
> > which is not in any *_ad_diff.list, so it's not very clear to me<br>
> > why it's needed.<br>
> ><br>
> > The other thing is that within streamice_init_fixed.f, there are<br>
> > some S/R calls that TAF does not know about, so that we now get<br>
> > (with recent version of TAF) this type of warning:<br>
> >> checking subroutine ** streamice_init_fixed **<br>
> >> *WARNING* : ad_input_code.f:XXX : call of streamice_init_diagnostics too many arguments (1) defined = 0<br>
> >> *WARNING* : ad_input_code.f:XXX : call of streamice_init_phi too many arguments (1) defined = 0<br>
> ><br>
> > Cheers,<br>
> > Jean-Michel<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><br>Daniel Goldberg, PhD<br>Lecturer in Glaciology<div>School of Geosciences, University of Edinburgh<br><span style="color:rgb(0,0,0);font-family:arial,helvetica,clean,sans-serif;line-height:15px">Geography Building, Drummond Street, Edinburgh EH8 9XP</span><br><br><br>em: <a href="mailto:dgoldber@mit.edu" target="_blank">D</a><a href="mailto:an.Goldberg@ed.ac.uk" target="_blank">an.Goldberg@ed.ac.uk</a><br>web: <a href="https://www.geos.ed.ac.uk/homes/dgoldber" target="_blank">https://www.geos.ed.ac.uk/homes/dgoldber</a></div></div></div></div></div>
</div>