<div dir="ltr">There are a lot of diffs like this. What make you think it is mine?  The cheby changes should not create such a large diff.<br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 24, 2016 at 3:23 PM, Satish Balay <span dir="ltr"><<a href="mailto:balay@mcs.anl.gov" target="_blank">balay@mcs.anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Another issue:<br>
<br>
<a href="http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/05/24/examples_next_arch-cuda-double_es.log" rel="noreferrer" target="_blank">http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/05/24/examples_next_arch-cuda-double_es.log</a><br>
<br>
/scratch/petsc/petsc.clone/src/ksp/ksp/examples/tests<br>
Possible problem with ex26_1, diffs above<br>
<br>
<br>
balay@asterix /home/balay/petsc (next=)<br>
$ git diff master..next -- src/ksp/ksp/examples/tests/output/ex26_1.out |cat<br>
diff --git a/src/ksp/ksp/examples/tests/output/ex26_1.out b/src/ksp/ksp/examples/tests/output/ex26_1.out<br>
index c46eca6..659624f 100644<br>
--- a/src/ksp/ksp/examples/tests/output/ex26_1.out<br>
+++ b/src/ksp/ksp/examples/tests/output/ex26_1.out<br>
@@ -1,10 +1,6 @@<br>
 Fine grid size 9 by 9<br>
-  0 KSP Residual norm 10.0656<br>
-  1 KSP Residual norm 4.35349<br>
-  2 KSP Residual norm 0.638099<br>
-  3 KSP Residual norm 0.0815887<br>
-  4 KSP Residual norm 0.00953547<br>
-  5 KSP Residual norm 0.00117851<br>
-  6 KSP Residual norm 0.000265924<br>
-  7 KSP Residual norm 2.74224e-05<br>
-Number of iterations = 7<br>
+  0 KSP Residual norm 23.5347<br>
+  1 KSP Residual norm 0.230059<br>
+  2 KSP Residual norm 0.000468129<br>
+  3 KSP Residual norm 1.16147e-06<br>
+Number of iterations = 3<br>
balay@asterix /home/balay/petsc (next=)<br>
$ git log --oneline src/ksp/ksp/examples/tests/output/ex26_1.out  |head -1<br>
7fd3bd7 updated test for new cheby<br>
balay@asterix /home/balay/petsc (next=)<br>
$ git branch -r --contains 7fd3bd7<br>
  origin/next<br>
balay@asterix /home/balay/petsc (next=)<br>
$<br>
<br>
So you made this change directly to next? Looks like next is way<br>
corrupted - and I should just rewind it to latest master..<br>
<span class="HOEnZb"><font color="#888888"><br>
Satish<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Tue, 24 May 2016, Satish Balay wrote:<br>
<br>
> On Tue, 24 May 2016, Mark Adams wrote:<br>
><br>
> > Do I now wait for my pull request to be approved?<br>
><br>
> Mark,<br>
><br>
> Currently there is a bit of mess in regards to your branches.<br>
><br>
> $ git branch -r |grep mark |grep ex56<br>
>   origin/mark/ksp-ex56<br>
>   origin/mark/ksp-newex56<br>
>   origin/mark/snes-ex56b<br>
>   origin/mark/snes-ex56c<br>
><br>
> You have a bunch of branches with the name 'ex56' - presumably you<br>
> worked initially on mark/snes-ex56b and switched to 'mark/snes-ex56c'.<br>
><br>
> And I see the the following 2 merges to next<br>
><br>
> >>>>>>><br>
> commit e223b1660d8274c4632642ec520eee8c34d18eb5<br>
> Merge: de2cc15 c13d8fd<br>
> Author: Barry Smith <<a href="mailto:bsmith@mcs.anl.gov">bsmith@mcs.anl.gov</a>><br>
> Date:   Mon May 23 15:27:49 2016 -0500<br>
><br>
>     Merge branch 'mark/snes-ex56b' into next<br>
><br>
> commit ebff70bc705bbe7fea86844e10d9a274abdf3658<br>
> Merge: c8d3fd4 dfc6d4a<br>
> Author: Satish Balay <<a href="mailto:balay@mcs.anl.gov">balay@mcs.anl.gov</a>><br>
> Date:   Thu May 19 23:59:01 2016 -0500<br>
><br>
>     Merge branch 'mark/snes-ex56c' into next<br>
> <<<<<<<<br>
><br>
> On my part - I noticed you had a buggy commit in next - which I wanted<br>
> to fix - so fixed in 'mark/snes-ex56c' and merged to next.<br>
><br>
> $ git branch -r --contains eb041292<br>
>   origin/mark/snes-ex56c<br>
>   origin/next<br>
><br>
> But how did this commit come to next? I don't see a prior 'Merge<br>
> branch 'mark/snes-ex56c' message in next history. Ok I think<br>
> c8d3fd49b4313a86346f23ef7d5371efba2d3dc9 was the next HEAD - when I<br>
> did the merge.<br>
><br>
> $ git log --oneline --merges --ancestry-path eb041292..c8d3fd49b4313a86346f23ef7d5371efba2d3dc9<br>
> c8d3fd4 Merge branch 'next' of bitbucket.org:petsc/petsc into next<br>
> 01528be Merge branch 'master' into next<br>
> 8bd3767 Merge branch 'pr476/psanan/psanan/pc-telescope-subcomm-type/master' into next<br>
> 83661ee Merge branch 'pr476/psanan/psanan/pc-telescope-subcomm-type/master' into next<br>
> 2a9b9f0 Merge branch 'master' into next<br>
> d5742a2 fixed error check in main of ex56<br>
> a003960 Merge branch 'master' of bitbucket.org:petsc/petsc into mark/snes-ex56c<br>
><br>
> So 'd5742a2 fixed error check in main of ex56' is a merge commit?<br>
><br>
> You did code change and merge in the same commit - and removed the<br>
> default git commit message - "Merge 'mark/snes-ex56c' into next"?<br>
><br>
> This is way confusing..<br>
><br>
> Now to figure out how to fix this. I see origin/mark/snes-ex56b is<br>
> merged to next - but origin/mark/snes-ex56c is not completely merged.<br>
><br>
> Is origin/mark/snes-ex56b abandoned? If so - I should revert Barry's<br>
> inadvertent merge of origin/mark/snes-ex56b into next - and merge<br>
> origin/mark/snes-ex56c<br>
><br>
> Thanks,<br>
> Satish<br>
><br>
<br>
</div></div></blockquote></div><br></div></div>