Skip to content

feat: point accessors#15

Merged
nblair2 merged 3 commits intomainfrom
feat/point-accessors
Apr 5, 2026
Merged

feat: point accessors#15
nblair2 merged 3 commits intomainfrom
feat/point-accessors

Conversation

@nblair2
Copy link
Copy Markdown
Owner

@nblair2 nblair2 commented Mar 29, 2026

Comprehensive re-write of the Point interface implementations to simplify the numerous different implementations to only a few based on their data types, and then add Get/Set functions for each of the fields that a Point may have.

@nblair2 nblair2 force-pushed the feat/point-accessors branch 2 times, most recently from 17b9825 to 94af0dd Compare March 29, 2026 19:49
@nblair2 nblair2 marked this pull request as ready for review March 29, 2026 23:04
@nblair2
Copy link
Copy Markdown
Owner Author

nblair2 commented Mar 29, 2026

Pending testing in downstream tools.

@nblair2
Copy link
Copy Markdown
Owner Author

nblair2 commented Mar 29, 2026

Ideally would not squash the 2 commits.

nblair2 added 2 commits April 3, 2026 10:01
Comprehensive re-write of the `Point` interface implementations to
simplify the numerous different implementations to only a few based
on their data types, and then add Get/Set functions for each of the
fields that a Point _may_ have.
@nblair2 nblair2 force-pushed the feat/point-accessors branch 3 times, most recently from 50795e8 to 2a268b3 Compare April 3, 2026 16:16
@nblair2 nblair2 force-pushed the feat/point-accessors branch from 2a268b3 to 7e5d12c Compare April 3, 2026 16:17
@nblair2 nblair2 merged commit 7ab8e2d into main Apr 5, 2026
3 checks passed
@nblair2 nblair2 deleted the feat/point-accessors branch April 5, 2026 01:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant