Skip to content

Feat/use calc direction#125

Merged
robitalec merged 140 commits intomainfrom
feat/use-calc-direction
Dec 16, 2025
Merged

Feat/use calc direction#125
robitalec merged 140 commits intomainfrom
feat/use-calc-direction

Conversation

@robitalec
Copy link
Copy Markdown
Member

@robitalec robitalec commented Oct 31, 2025

Using calc_direction throughout functions, see #116

For direction_step, direction_to_centroid, direction_to_leader and edge_direction:

  • provide alternative sfc interface for direction functions
  • if coords are provided, ensure crs is provided (required for lwgeom::st_geod_azimuth)
  • if coords are NULL, use default "geometry" sfc interface
  • use env argument for programming on data.tables (Fix: Use data.table's env argument #95)

Will require tests added for direction_to_centroid and direction_to_leader after calc_centroid is used throughout (#127)

@robitalec robitalec self-assigned this Oct 31, 2025
@robitalec robitalec added the type: enhancement new features, improvements label Oct 31, 2025
@robitalec robitalec force-pushed the feat/use-calc-direction branch from 5e0a48f to 0f4c2ae Compare November 6, 2025 18:47
@robitalec robitalec merged commit 21de1f2 into main Dec 16, 2025
7 of 8 checks passed
@robitalec robitalec deleted the feat/use-calc-direction branch December 16, 2025 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: enhancement new features, improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant