<div dir="ltr">On Thu, Oct 24, 2013 at 10:31 AM, Jed Brown <span dir="ltr"><<a href="mailto:jedbrown@mcs.anl.gov" target="_blank">jedbrown@mcs.anl.gov</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">@madams This branch is totally unreviewable in its current form, and<br>

it cannot be bisected because many of the intermediate states are<br>
broken.  I understand making haphazard/checkpoint commits while<br>
developing, but to maintain reviewability and quality in the long term,<br>
it's important to organize the commits so that they can be understood in<br>
sequence.  Commit messages like "...", "temp version", and "cleaning up"<br>
force the reviewer to read the commit in its entirety to have any idea<br>
what it is supposed to accomplish.  This makes it difficult to keep<br>
track of what is happening, which discourages people from following<br>
development, avoiding duplicate/conflicting work, etc.<br>
<br>
We have to support everything that goes into PETSc, but if we can't<br>
follow development, we give outdated advice and end up unable to answer<br>
questions.<br>
<br>
Most people working on PETSc are organizing their commits to be<br>
reviewable these days.  We can go to the commit and branch lists (in the<br>
web interface or with Git locally) and the first line of the commit<br>
message gives a good summary of what the commit is accomplishing, the<br>
body of the commit message explains why it is important/who may be<br>
impacted by the change, and the commit itself accomplishes something<br>
that the viewer can check.  This allows other developers to quickly get<br>
the gist of what is in a branch, what may be interesting to look at in<br>
more detail, etc.<br>
<br>
Commit topology and messages are key means by which developers<br>
communicate with each other.  It really doesn't take much effort once<br>
you get in the habit of organizing commits as a means of communication<br>
rather than as a log with arbitrary checkpoints.<br>
<br>
Any other means of communication, such as pull request summaries or<br>
mailing list threads, are secondary sources, inherently more error-prone<br>
and higher effort to evaluate, and provide less structure for automated<br>
diagnostics/summaries.<br>
<br>
<a href="https://bitbucket.org/petsc/petsc/wiki/writing-commit-messages" target="_blank">https://bitbucket.org/petsc/petsc/wiki/writing-commit-messages</a><br>
<br>
Git provides good tools for reordering/squashing/amending commits, so<br>
it's possible to develop in clutter and then organize when it's time to<br>
communicate to other people.<br>
<br>
<br>
This PR is blocked on some of Matt's long-running branches.  @knepley<br>
When will 'knepley/feature-dmda-section' and<br>
'knepley/feature-plex-refine-3d' be ready to merge?<br></blockquote><div><br></div><div>knepley/feature-plex-refine-3d this could really merge tomorrow if we needed it, and<br></div><div>I would do another branch to add the hex support.</div>
<div><br></div><div>It should not be blocked on knepley/feature-dmda-section, which is going to take some</div><div>time.</div><div><br></div><div>   Matt</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

Mark Adams <<a href="mailto:pullrequests-reply@bitbucket.org">pullrequests-reply@bitbucket.org</a>> writes:<br>
<br>
> --- you can reply above this line ---<br>
><br>
> A new pull request has been opened by Mark Adams.<br>
><br>
> madams/sr-driver2 has changes to be pulled into master.<br>
><br>
> <a href="https://bitbucket.org/petsc/petsc/pull-request/109/added-at-test-simplified-version-of-ex11" target="_blank">https://bitbucket.org/petsc/petsc/pull-request/109/added-at-test-simplified-version-of-ex11</a><br>

><br>
> Title: Added at test, simplified version of ex11, convergence test.  Uses differencing to produce matrix in stand alone code, fixed SNES to support this.<br>
><br>
> Changes to be pulled:<br>
><br>
> c1fa6a8934f2 by Mark Adams: "cleanup jed's cherry picked version"<br>
> e7b0c4510d02 by Jed Brown: "Merge branch 'knepley/fix-plex-ghost-cells' into jed/sr-driver<br>
><br>
> DMLabelFilter is…"<br>
> c56f055af68b by Mark Adams: "finished up adding serial DMPlex test with a convergance test on a 5-point Lapla…"<br>
> 8fdc87fb42d2 by Mark Adams: "cleaning up"<br>
> 0fb202fbc65d by Mark Adams: "cleaning up<br>
><br>
> Conflicts:<br>
>       src/ts/examples/tutorials/makefile"<br>
> ... and 112 more.<br>
><br>
><br>
> --<br>
<div class="im">><br>
> Unsubscribe from pull request emails for this repository.<br>
</div>> <a href="https://bitbucket.org/petsc/petsc/pull-request/109/unsubscribe/jedbrown/bc8a9c9283892888c102096c5dadc4a6a692a4fe/" target="_blank">https://bitbucket.org/petsc/petsc/pull-request/109/unsubscribe/jedbrown/bc8a9c9283892888c102096c5dadc4a6a692a4fe/</a><br>

</blockquote></div><br><br clear="all"><div><br></div>-- <br>What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>
-- Norbert Wiener
</div></div>