fork/Graphite
1
0
Fork 0
mirror of https://github.com/GraphiteEditor/Graphite.git synced 2026-09-26 08:18:12 +08:00
Code Issues Packages Projects Releases Wiki Activity

Make the 'Index Points' node index within each vector element instead of across the whole list (#4427)

Browse Source
This commit is contained in:
Keavon Chambers
2026-09-15 14:40:26 +02:00
committed by Dennis Kobert
parent 29540adc8f
commit 0e0687c8cb
1 changed files with 11 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files
node-graph/nodes/vector/src/vector_nodes.rs
+11 -21
View File
@@ -3835,41 +3835,31 @@ fn count_points(_: impl Ctx, content: IList<Vector>) -> f64 {
(0..content.len()).map(|index| content.element_ref(index).point_domain.positions().len() as f64).sum() (0..content.len()).map(|index| content.element_ref(index).point_domain.positions().len() as f64).sum()
} }
/// Retrieves the vec2 position (in local space) of the anchor point at the specified index in a `List` of vector elements. /// Retrieves the vec2 position (in local space) of the anchor point at the specified index within a vector element.
/// If no value exists at that index, the position (0, 0) is returned. /// If no value exists at that index, the position (0, 0) is returned.
#[node_macro::node(category("Vector: Measure"), path(graphene_core::vector))] #[node_macro::node(category("Vector: Measure"), path(graphene_core::vector))]
fn index_points( fn index_points(
_: impl Ctx, _: impl Ctx,
/// The vector element or elements containing the anchor points to be retrieved. /// The vector element containing the anchor points to be retrieved.
content: IList<Vector>, content: Vector,
/// The index of the points to retrieve, starting from 0 for the first point. Negative indices count backwards from the end, starting from -1 for the last item. /// The index of the points to retrieve, starting from 0 for the first point. Negative indices count backwards from the end, starting from -1 for the last point.
index: f64, index: f64,
) -> DVec2 { ) -> DVec2 {
let points_count = (0..content.len()).map(|row| content.element_ref(row).point_domain.positions().len()).sum::<usize>(); let positions = content.point_domain.positions();
if points_count == 0 { let Some(last_index) = positions.len().checked_sub(1) else {
return DVec2::ZERO; return DVec2::ZERO;
} };
// Clamp and allow negative indexing from the end // Clamp and allow negative indexing from the end
let index = index as isize; let index = index as isize;
let index = if index < 0 { let index = if index < 0 {
(points_count as isize + index).max(0) as usize (positions.len() as isize + index).max(0) as usize
} else { } else {
(index as usize).min(points_count - 1) (index as usize).min(last_index)
}; };
// Find the point at the given index across all vector elements positions[index]
let mut accumulated = 0;
for row in 0..content.len() {
let vector = content.element_ref(row);
let row_point_count = vector.point_domain.positions().len();
if index - accumulated < row_point_count {
return vector.point_domain.positions()[index - accumulated];
}
accumulated += row_point_count;
}
DVec2::ZERO
} }
#[node_macro::node(category("Vector: Measure"), path(core_types::vector))] #[node_macro::node(category("Vector: Measure"), path(core_types::vector))]