Skip to content

ENH: Initialize PolyLineCell with 2 points#3409

Merged
hjmjohnson merged 1 commit intoInsightSoftwareConsortium:masterfrom
PranjalSahu:pranjal15
May 4, 2022
Merged

ENH: Initialize PolyLineCell with 2 points#3409
hjmjohnson merged 1 commit intoInsightSoftwareConsortium:masterfrom
PranjalSahu:pranjal15

Conversation

@PranjalSahu
Copy link
Contributor

@PranjalSahu PranjalSahu commented May 2, 2022

PolyLine Cell should have at least two points so inserting those points in the default constructor.
For code re-use added InitializePoints method that is used in constructor and ClearPoints method.

PR Checklist

  • No API changes were made (or the changes have been approved)
  • No major design changes were made (or the changes have been approved)
  • Added test (or behavior not changed)
  • Updated API documentation (or API not changed)
  • Added license to new files (if any)
  • Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
  • Added ITK examples for all new major features (if any)

Refer to the ITK Software Guide for
further development details if necessary.

@github-actions github-actions bot added area:Core Issues affecting the Core module type:Enhancement Improvement of existing methods or implementation labels May 2, 2022
@PranjalSahu PranjalSahu mentioned this pull request May 2, 2022
7 tasks
@PranjalSahu PranjalSahu changed the title ENH: Initialize PolyLine Cell with 2 points ENH: Initialize PolyLineCell with 2 points May 3, 2022
Atleast two points should be present in a PolyLineCell so inserting
2 points in the default constructor.
@hjmjohnson hjmjohnson merged commit 5b00720 into InsightSoftwareConsortium:master May 4, 2022
@PranjalSahu PranjalSahu deleted the pranjal15 branch July 19, 2022 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:Core Issues affecting the Core module type:Enhancement Improvement of existing methods or implementation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants