<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    Hi Andy,<br>
    <br>
    I followed the directions, and submitted a patch (19 days ago), but
    haven't heard anything back.<br>
    <br>
    In GitLab the merge requests counter is at 0, but when I try to
    create a new merge request from my commit, it tells the merge
    request already exists. Is it being reviewed somewhere? Thanks!<br>
    <br>
    Martin<br>
    <br>
    <div class="moz-cite-prefix">On 29/12/2015 13:22, Andy Bauer wrote:<br>
    </div>
    <blockquote
cite="mid:CAMaOp+GpHqJvvuX+SJNKd7qr=zt5yotwumcMLVwk_pMtOOV5JA@mail.gmail.com"
      type="cite">
      <div dir="ltr">
        <div>
          <div>
            <div>Hi Martin,<br>
              <br>
            </div>
            This patch makes sense. It would need a test if you want to
            get your changes into VTK. The directions for contributing
            to VTK are at <a moz-do-not-send="true"
href="https://gitlab.kitware.com/vtk/vtk/blob/master/Documentation/dev/git/develop.md">https://gitlab.kitware.com/vtk/vtk/blob/master/Documentation/dev/git/develop.md</a>.<br>
            <br>
          </div>
          Cheers,<br>
        </div>
        Andy<br>
      </div>
      <div class="gmail_extra"><br>
        <div class="gmail_quote">On Mon, Dec 28, 2015 at 5:50 PM, Martin
          Genet <span dir="ltr"><<a moz-do-not-send="true"
              href="mailto:martin.genet@polytechnique.edu"
              target="_blank">martin.genet@polytechnique.edu</a>></span>
          wrote:<br>
          <blockquote class="gmail_quote" style="margin:0 0 0
            .8ex;border-left:1px #ccc solid;padding-left:1ex">
            <div text="#000000" bgcolor="#FFFFFF"> Thanks Andy.<br>
              <br>
              What about simply adding another mode, e.g.
              SetTensorModeToComputeGreenLagrangeStrain, to the
              vtkCellDerivatives filter? Would the attached patch make
              sense?<br>
              <br>
              Martin<br>
              <br>
              <div>On 28/12/2015 14:12, Andy Bauer wrote:<br>
              </div>
              <blockquote type="cite">
                <div dir="ltr">
                  <div>
                    <div>
                      <div>Hi Martin,<br>
                        <br>
                      </div>
                      Changing the name of the
                      SetTensorModeToComputeStrain method to something
                      else would break backward compatibility which is
                      generally avoided in VTK. Other options for this
                      include deriving a class to compute non-linear
                      strain from vtkCellDerivatives if it shares enough
                      of the algorithm with the linearized version or
                      maybe just creating a new filter.<br>
                      <br>
                    </div>
                    Cheers,<br>
                  </div>
                  Andy<br>
                </div>
                <div class="gmail_extra"><br>
                  <div class="gmail_quote">On Sat, Dec 26, 2015 at 4:36
                    PM, Martin Genet <span dir="ltr"><<a
                        moz-do-not-send="true"
                        href="mailto:martin.genet@polytechnique.edu"
                        target="_blank"><a class="moz-txt-link-abbreviated" href="mailto:martin.genet@polytechnique.edu">martin.genet@polytechnique.edu</a></a>></span>
                    wrote:<br>
                    <blockquote class="gmail_quote" style="margin:0 0 0
                      .8ex;border-left:1px #ccc solid;padding-left:1ex">Dear
                      VTK users:<br>
                      <br>
                      I realize that the vtkCellDerivatives filter, when
                      SetTensorModeToComputeStrain is activated, returns
                      the symmetric part of the gradient of the input
                      vector field, which is the linearized strain
                      tensor, i.e., not a proper measure of deformation
                      when large displacements are involved. Would that
                      make sense to have two different modes,
                      SetTensorModeToComputeLinearizedStrain or
                      SetTensorModeToComputeSymmetricGradient, and
                      SetTensorModeToComputeStrain or
                      SetTensorModeToComputeGreenLagrangeStrain? Thanks!<br>
                      <br>
                      Martin<br>
                      _______________________________________________<br>
                      Powered by <a moz-do-not-send="true"
                        href="http://www.kitware.com" rel="noreferrer"
                        target="_blank">www.kitware.com</a><br>
                      <br>
                      Visit other Kitware open-source projects at <a
                        moz-do-not-send="true"
                        href="http://www.kitware.com/opensource/opensource.html"
                        target="_blank"><a class="moz-txt-link-freetext" href="http://www.kitware.com/opensource/opensource.html">http://www.kitware.com/opensource/opensource.html</a></a><br>
                      <br>
                      Please keep messages on-topic and check the VTK
                      FAQ at: <a moz-do-not-send="true"
                        href="http://www.vtk.org/Wiki/VTK_FAQ"
                        target="_blank">http://www.vtk.org/Wiki/VTK_FAQ</a><br>
                      <br>
                      Search the list archives at: <a
                        moz-do-not-send="true"
                        href="http://markmail.org/search/?q=vtkusers"
                        rel="noreferrer" target="_blank"><a class="moz-txt-link-freetext" href="http://markmail.org/search/?q=vtkusers">http://markmail.org/search/?q=vtkusers</a></a><br>
                      <br>
                      Follow this link to subscribe/unsubscribe:<br>
                      <a moz-do-not-send="true"
                        href="http://public.kitware.com/mailman/listinfo/vtkusers"
                        rel="noreferrer" target="_blank">http://public.kitware.com/mailman/listinfo/vtkusers</a><br>
                    </blockquote>
                  </div>
                  <br>
                </div>
              </blockquote>
              <br>
            </div>
          </blockquote>
        </div>
        <br>
      </div>
    </blockquote>
    <br>
  </body>
</html>