[MITgcm-devel] After PR #371 (tst_2+2_bug_fix) got merged in

[Edward Doddridge]

The cleanest solution is to make a new branch off `master` and new pr with that branch.

Ed


From: MITgcm-devel <mitgcm-devel-bounces at mitgcm.org> on behalf of Martin Losch <Martin.Losch at awi.de>
Reply to: "MITgcm-devel at mitgcm.org" <MITgcm-devel at mitgcm.org>
Date: Thursday, 24 September 2020 at 16:03
To: MITgcm-devel <MITgcm-devel at mitgcm.org>
Subject: Re: [MITgcm-devel] After PR #371 (tst_2+2_bug_fix) got merged in

Hi,

Oliver is right, the solution on ollie and on my MacOS with POSIX sed is to remove the first blank in the new line ***before*** the “\”. The blank after the backslash is needed to that it’s not interpreted as “\n”

How do I do this now. New branch with new PR or do I use the old branch, add the fix and make a new PR?

Martin

> On 23. Sep 2020, at 17:04, Oliver Jahn <jahn at mit.edu> wrote:
>
> Looks like this:
>
> sed -e "/^ *\&PARM03/a\\
> \ nTimeSteps=$Dbl," data.tst > data.tmp_$$
>
> introduces 2 spaces before "nTimeSteps" which the trips up a later part
> of the script:
>
> Dbl=`sed -n 's/^ nTimeSteps=//p' data.tst | sed 's/,//g'`
>
> Remove the space before the backslash in the new sed command? I assume
> the escaped space is somehow needed for POSIX???
>
> Oliver
>
>
> On 2020-09-23 10:45, Martin Losch wrote:
>> That’s indeed interesting. This can only mean that the sed commands are not always doing, what they are supposed to do. I cannot inspect this right now, but will do so asap for ollie. I did test this for both POSIX sed and GNU sed.
>>
>> Martin
>>
>>> On 23. Sep 2020, at 16:12, Jean-Michel Campin <jmc at mit.edu> wrote:
>>>
>>> Hi Martin and Oliver,
>>>
>>> There is something unclear regarding the effect of this PR #371:
>>> Since I merged it after mid-night yesterday, only got used in faily tests
>>> last night. And for few restart test where some experiments were failing the
>>> test, I got a big improvement and many more to pass.
>>>
>>> I still have not yet all output back, but the one affected are:
>>> a) 1 more pass on villon (linux_amd64_gfortran.dvlp):
>>> Y Y Y Y pass <- global_ocean.gm_k3d
>>> ---
>>>> Y Y Y Y FAIL (4) - global_ocean.gm_k3d
>>> b) 5 more pass on engaging (linux_amd64_open64+mpi.fast):
>>> 45c45
>>> < Y Y Y Y pass <- global_with_exf
>>> ---
>>>> Y Y Y Y FAIL (4) - global_with_exf
>>> 56,59c56,59
>>> < Y Y Y Y pass <- isomip
>>> < Y Y Y Y pass <- isomip.htd
>>> < Y Y Y Y pass <- isomip.icefront
>>> < Y Y Y Y pass <- isomip.obcs
>>> ---
>>>> Y Y Y Y FAIL (4) - isomip
>>>> Y Y Y Y FAIL (4) - isomip.htd
>>>> Y Y Y Y FAIL (4) - isomip.icefront
>>>> Y Y Y Y FAIL (4) - isomip.obcs
>>> c) and the largest changes are from "ollie" (before/after)i, only the "fast" ones:
>>> 46:97 rs_ollie-i_20200922_1 restart linux_ia64_ifort_ollie+mpi.fast
>>> 94:97 rs_ollie-i_20200923_1 restart linux_ia64_ifort_ollie+mpi.fast
>>> And:
>>> 50:96 rs_ollie-c_20200922_1 restart linux_ia64_cray_ollie+mpi.fast
>>> 92:96 rs_ollie-c_20200923_1 restart linux_ia64_cray_ollie+mpi.fast
>>>
>>> The 1 change in "do_tst_2+2" is unlikely to change anything (because it's just
>>> for log file and report, and only excecuted once for the first experiment).
>>> So it's something with the 2 changes in "tst_2+2", and may be related to
>>> disk/system missing an updated file (since it's more common with "fast" than "devel") ?
>>> Not very clear.
>>>
>>> Cheers,
>>> Jean-Michel
>>> _______________________________________________
>>> MITgcm-devel mailing list
>>> MITgcm-devel at mitgcm.org
>>> http://mailman.mitgcm.org/mailman/listinfo/mitgcm-devel<http://mailman.mitgcm.org/mailman/listinfo/mitgcm-devel>
>>
>> _______________________________________________
>> MITgcm-devel mailing list
>> MITgcm-devel at mitgcm.org
>> http://mailman.mitgcm.org/mailman/listinfo/mitgcm-devel<http://mailman.mitgcm.org/mailman/listinfo/mitgcm-devel>
>>
> _______________________________________________
> MITgcm-devel mailing list
> MITgcm-devel at mitgcm.org
> http://mailman.mitgcm.org/mailman/listinfo/mitgcm-devel<http://mailman.mitgcm.org/mailman/listinfo/mitgcm-devel>

_______________________________________________
MITgcm-devel mailing list
MITgcm-devel at mitgcm.org
http://mailman.mitgcm.org/mailman/listinfo/mitgcm-devel<http://mailman.mitgcm.org/mailman/listinfo/mitgcm-devel>



University of Tasmania Electronic Communications Policy (December, 2014).
This email is confidential, and is for the intended recipient only. Access, disclosure, copying, distribution, or reliance on any of it by anyone outside the intended recipient organisation is prohibited and may be a criminal offence. Please delete if obtained in error and email confirmation to the sender. The views expressed in this email are not necessarily the views of the University of Tasmania, unless clearly intended otherwise.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.mitgcm.org/pipermail/mitgcm-devel/attachments/20200924/ef109ba1/attachment-0001.html>