Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Line semantics #1334

Merged
merged 4 commits into from
Jul 31, 2024
Merged

Line semantics #1334

merged 4 commits into from
Jul 31, 2024

Conversation

julianschuler
Copy link
Member

Why? What?

This redefines a line as a point and a direction vector to further emphasize the difference to a line segment.

It additionally adds test cases for all functions defined for it.

ToDo / Known Issues

None.

Ideas for Next Iterations (Not This PR)

None.

How to Test

Build the code, run the tests.

@julianschuler julianschuler added the is:Refactoring No changes in functionality, only in coding style. label Jul 18, 2024
@julianschuler julianschuler force-pushed the line-semantic branch 2 times, most recently from 6b046c8 to f02ebb2 Compare July 18, 2024 17:54
@schmidma schmidma self-assigned this Jul 19, 2024
crates/geometry/src/line.rs Show resolved Hide resolved
crates/geometry/src/line.rs Show resolved Hide resolved
crates/geometry/src/line.rs Outdated Show resolved Hide resolved
@julianschuler julianschuler added this pull request to the merge queue Jul 31, 2024
Merged via the queue into HULKs:main with commit b70ffa3 Jul 31, 2024
25 checks passed
@julianschuler julianschuler deleted the line-semantic branch July 31, 2024 13:30
oleflb pushed a commit to oleflb/hulk that referenced this pull request Oct 7, 2024
* Define line as a point and direction vector

* Add tests for line

* Add comment explaining manual default implementation

* Redefine Partial/Relative/AbsDiff equalities

---------

Co-authored-by: Julian Schuler <[email protected]>
pejotejo pushed a commit to pejotejo/hulk that referenced this pull request Nov 6, 2024
* Define line as a point and direction vector

* Add tests for line

* Add comment explaining manual default implementation

* Redefine Partial/Relative/AbsDiff equalities

---------

Co-authored-by: Julian Schuler <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is:Refactoring No changes in functionality, only in coding style.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants