<div dir="ltr">Hi Dr. Thompson,<div><br></div><div>Sorry for late reply. Since the midterm is coming, I'll definitely speed up the development from now on. I have a question about the class design in your last email. To my understanding, control points for one patch is stored in a vtkDataArray. Do you mean store multiple vtkDataArray regarding to multiple patches in the vtkDataObject? If so, what class do I need to use for vtkDataObject?</div><div><br></div><div>Best,</div><div>Lin</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 10, 2015 at 2:39 PM, David Thompson <span dir="ltr"><<a href="mailto:david.thompson@kitware.com" target="_blank">david.thompson@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Lin,<br>
<br>
> I have made it support 3D ellipse with function ...<br>
<br>
Great!<br>
<span class=""><br>
> As you can see in the python test, I call this function 4 times for each quadrant to generate the entire ellipse. Since we need to support multiple patches, I think it's better to define a new dataset class for bezier patches as you mentioned before about  creating a new vtkControlPoints class that inherits from vtkPoints and allows for an extra coordinate.<br>
<br>
</span>Rather than create new dataset types, I would like to use existing datatypes to store the data and make an adaptor to iterate over them. That way existing pipelines could modify scalar values defined on control points (and perhaps even patches). For instance, B-splines could be represented as a vtkStructuredGrid whose FieldData contains a special knot array (special in the sense that the array must be of the proper size and with a fixed name like "_vtkKnotVector").<br>
<br>
The adaptor would take a vtkDataObject as input and provide iterator-style access to each patch defined by the dataset. Specific subclasses of the adaptor would be B-splines (which expect the vtkDataObject to be a vtkStructuredGrid) or T-splines (which might expect a vtkUniformGridAMR) or even point-based splines (PB-splines as defined in Sederberg's T-spline paper, which could expect any dataset derived vtkPointSet).<br>
<br>
class vtkBezierPatchAdaptor : public vtkObject<br>
{<br>
public:<br>
  virtual void SetControlPointData(vtkDataObject* controlPointData);<br>
  vtkGetObjectMacro(vtkDataObject,ControlPointData);<br>
<br>
  // Methods to iterate over patches:<br>
  virtual void Begin() = 0;<br>
  virtual bool IsAtEnd() = 0;<br>
  virtual bool GoToPatch(vtkIdType patchId) = 0; // random access iteration<br>
  virtual bool Next() = 0;<br>
<br>
  // Methods to access the current patch:<br>
  virtual int GetPatchDimension() const = 0;<br>
  virtual void GetPatchShape() const = 0;<br>
    // returns VTK_TRIANGLE/VTK_QUAD when PatchDimension==2,<br>
    // returns VTK_HEXAHEDRON/VTK_TETRA when PatchDimension==3<br>
  virtual void GetPatchDegree(int* degreeOut) const = 0;<br>
  virtual void GetPatchParameterRange(double* paramsOut) const = 0;<br>
  virtual void GetPatchPoints(vtkPoints* pointsOut) const = 0;<br>
<br>
  // Some helper methods that would make Python wrappings usable:<br>
  int GetPatchDegree(int dim) const<br>
    {<br>
    std::vector<int> degree(this->GetPatchDimension());<br>
    this->GetPatchDegree(&degree[0]);<br>
    return degree[dim];<br>
    }<br>
  void GetPatchParameterRange(int dim, double paramRange[2])<br>
    {<br>
    std::vector<double> range(2 * this->GetPatchDimension());<br>
    this->GetPatchDegree(&range[0]);<br>
    paramRange[0] = [2 * dim];<br>
    paramRange[1] = [2 * dim + 1];<br>
    }<br>
<br>
protected:<br>
  vtkDataObject* ControlPointData;<br>
};<br>
<br>
The B-spline subclass could then iterate over patches by keeping a current "cell ID," which would be incremented by calling Next(). When asked for the *patch* points (not the input points), the adaptor would create a vtkMappedDataArray (see <a href="http://www.vtk.org/Wiki/VTK/InSituDataStructure" rel="noreferrer" target="_blank">http://www.vtk.org/Wiki/VTK/InSituDataStructure</a> for more information) that did not copy control point coordinates from its ControlPointData->GetPoints() but instead used the implicit connectivity to provide access to underlying B-spline points.<br>
<br>
For example, consider the simple case of a rectangular B-spline with degree 1 (bi-linear). We might be given a 5x6 grid of control points as input and asked to iterate over all the patches. Each patch is simply a quadrilateral, and there are (5-1)*(6-1) = 20 of them.<br>
<br>
  adaptor->Begin(); // would initialize an internal "cell ID" to 0<br>
  adaptor->IsAtEnd(); // would return false for "cell ID" values in [0,19] and true otherwise.<br>
  adaptor->GetPatchDimension(); // would return 2<br>
  adaptor->GetPatchShape(); // would return VTK_QUAD<br>
  adaptor->GetPatchDegree(degreeOut); // would populate degreeOut with [1,1]<br>
  adaptor->GetPatchPoints(pointsOut); // would populate pointsOut with a vtkMappedDataArray.<br>
<br>
The vtkMappedDataArray would report having 4 tuples (one for each corner of the quad... when the degree is higher, is would report the product of (degreeOut[i]+1) for all i in GetPatchDimension()). The actual points reported would depend on the "cell ID" in the adaptor.<br>
<br>
I have not figured out yet how the "w" homogeneous coordinate would work with vtkPoints. A different adaptor API might work better (keeping the "w" coordinate separate from the other points).<br>
<span class=""><br>
> Another question is that currently the domain of parameter coordinate is [0, 1] for each patch. Do you think it's better to make the domain continuously, namely [0, 0.25] for the first quadrant, [0.25, 0.75] for the second quadrant, etc?<br>
<br>
</span>If we focus on the B-spline adaptor above, we get that for free since the returned knot vector could be normalized to the range [0,1].<br>
<span class="HOEnZb"><font color="#888888"><br>
        David</font></span></blockquote></div><br></div>