griefith/test2
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Fix #132537: Assert when baking with string attribute

Browse Source 
Propagating string attributes isn't supported in many places currently,
the split edges node is not an exception, so skip those attributes as
necessary.
This commit is contained in:
Hans Goudey
2025-01-14 22:29:03 -05:00
parent ac64ada636
commit 98060efcf1
1 changed files with 10 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
source/blender/geometry/intern/mesh_split_edges.cc
View File

@@ -29,18 +29,20 @@ static void propagate_vert_attributes(Mesh &mesh, const Span<int> new_to_old_ver
bke::MutableAttributeAccessor attributes = mesh.attributes_for_write();
for (const StringRef id : attributes.all_ids()) {
if (attributes.lookup_meta_data(id)->domain != bke::AttrDomain::Point) {
const bke::AttributeMetaData meta_data = *attributes.lookup_meta_data(id);
if (meta_data.domain != bke::AttrDomain::Point) {
continue;
}
if (meta_data.data_type == CD_PROP_STRING) {
continue;
}
bke::GSpanAttributeWriter attribute = attributes.lookup_for_write_span(id);
if (!attribute) {
continue;
}
bke::attribute_math::gather(attribute.span,
new_to_old_verts_map,
attribute.span.take_back(new_to_old_verts_map.size()));
attribute.finish();
}
if (float3 *orco = static_cast<float3 *>(
@@ -68,7 +70,11 @@ static void propagate_edge_attributes(Mesh &mesh, const Span<int> new_to_old_edg
bke::MutableAttributeAccessor attributes = mesh.attributes_for_write();
for (const StringRef id : attributes.all_ids()) {
if (attributes.lookup_meta_data(id)->domain != bke::AttrDomain::Edge) {
const bke::AttributeMetaData meta_data = *attributes.lookup_meta_data(id);
if (meta_data.domain != bke::AttrDomain::Edge) {
continue;
}
if (meta_data.data_type == CD_PROP_STRING) {
continue;
}
if (id == ".edge_verts") {