Hans Goudey
7484f274dc
This commit changes the Curve to Mesh node to work with `Curves` instead of `CurveEval`. The change ends up basically completely rewriting the node, since the different attribute storage means that the decisions made previously don't make much sense anymore. The main loops are now "for each attribute: for each curve combination" rather than the other way around, with the goal of taking advantage of the locality of curve attributes. This improvement is quite noticeable with many small curves; I measured a 4-5x improvement (around 4-5s to <1s) when converting millions of curves to tens of millions of faces. I didn't obverse any change in performance compared to 3.1 with fewer curves though. The changes also solve an algorithmic flaw where any interpolated attributes would be evaluated for every curve combination instead of just once per curve. This can be a large improvement when there are many profile curves. The code relies heavily on a function `foreach_curve_combination` which calculates some basic information about each combination and calls a templated function. I made assumptions about unnecessary reads being removed by compiler optimizations. For further performance improvements in the future that might be an area to investigate. Another might be using a "for a group of curves: for each attribute: for each curve" pattern to increase the locality of memory access. Differential Revision: https://developer.blender.org/D14642
34 lines
1.1 KiB
C++
34 lines
1.1 KiB
C++
/* SPDX-License-Identifier: GPL-2.0-or-later */
|
|
|
|
#pragma once
|
|
|
|
struct CurvesGeometry;
|
|
struct Mesh;
|
|
|
|
/** \file
|
|
* \ingroup bke
|
|
*/
|
|
|
|
namespace blender::bke {
|
|
|
|
/**
|
|
* Extrude all splines in the profile curve along the path of every spline in the curve input.
|
|
* Transfer curve attributes to the mesh.
|
|
*
|
|
* \note Normal calculation is by far the slowest part of calculations relating to the result mesh.
|
|
* Although it would be a sensible decision to use the better topology information available while
|
|
* generating the mesh to also generate the normals, that work may wasted if the output mesh is
|
|
* changed anyway in a way that affects the normals. So currently this code uses the safer /
|
|
* simpler solution of deferring normal calculation to the rest of Blender.
|
|
*/
|
|
Mesh *curve_to_mesh_sweep(const CurvesGeometry &main,
|
|
const CurvesGeometry &profile,
|
|
bool fill_caps);
|
|
/**
|
|
* Create a loose-edge mesh based on the evaluated path of the curve's splines.
|
|
* Transfer curve attributes to the mesh.
|
|
*/
|
|
Mesh *curve_to_wire_mesh(const CurvesGeometry &curve);
|
|
|
|
} // namespace blender::bke
|