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.

Sign up for GitHub

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

Jump to bottom

Add Line Segment primitive geometry class #33

Closed
chadqueen opened this issue Aug 31, 2022 · 3 comments · Fixed by #54
Closed

Add Line Segment primitive geometry class #33

chadqueen opened this issue Aug 31, 2022 · 3 comments · Fixed by #54
Assignees
@RobPasMue
Labels
PRIORITY Issue/PR assigned with this tag must be given priority over the rest of the backlog

Comments

@chadqueen
Copy link
Contributor

chadqueen commented Aug 31, 2022 •
edited
Loading

Minimum constructor
Point3D start, Point3D end
@chadqueen chadqueen mentioned this issue Aug 31, 2022
Closed
27 tasks
@chadqueen chadqueen self-assigned this Aug 31, 2022
@RobPasMue
Copy link
Member

This already exists - It is the actual LineSketch we have

@RobPasMue
Copy link
Member

LineSegment should extend from Line. Solve #34 first

@github-actions github-actions bot removed the PRIORITY label Sep 5, 2022
@RobPasMue RobPasMue assigned RobPasMue and unassigned chadqueen Sep 5, 2022
@RobPasMue
Copy link
Member

Working on #34, #33 and #43 at the same time

This was referenced Sep 5, 2022
Closed
Closed
Merged
@RobPasMue RobPasMue added the PRIORITY Issue/PR assigned with this tag must be given priority over the rest of the backlog label Sep 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@RobPasMue RobPasMue

Labels
PRIORITY Issue/PR assigned with this tag must be given priority over the rest of the backlog
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Implementing Line and Segment classes ansys/pyansys-geometry
2 participants
@chadqueen @RobPasMue