Working with segments - vignette #66

Activity

1. Robinlovelace commented on Jun 24, 2025

   

   Robinlovelace

   on Jun 24, 2025

   Member

   More actions

   Agreed, all segments the same colour, right? 🧐

   👍React with 👍1Reacted by RFlx
2. temospena commented on Jun 16, 2026

   

   temospena

   on Jun 16, 2026

   CollaboratorAuthor

   More actions

   the plot code at L173 was fine — the real bug was in elevation_add(). It used m[, 3] (positional) to split the coordinate matrix by feature, but sf::st_coordinates() on a cast-segmentised sf returns columns X, Y, L1 — so the positional index 3 is L1, which starts at 1 for every feature. All segments end up merged into one geometry, slope_xyz gets one group → one scalar → identical slope for all rows.

   Fix: m[, "L1"] — uses the named column, which correctly holds the feature row index.
3. temospena

   added 3 commits that reference this issue on Jun 16, 2026

   #66 update column index reference from numeric to L1 in elevation_add()

   36cec5f

   fix vignette with linestring break segments and add stplanar example …

   ...

   753ec37

   #66 add route_to_segments function to split linestrings into vertex-t…

   ...

   9eb8246
4. temospena

   linked a pull request that will close this issue66 working with segments vignette #76on Jun 16, 2026
5. temospena

   closed this as completedin #76on Jun 16, 2026