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

Add functionality to resample points along a curve #11

Open
tavaughan opened this issue Oct 27, 2017 · 1 comment
Open

Add functionality to resample points along a curve #11

tavaughan opened this issue Oct 27, 2017 · 1 comment
Assignees

Comments

@tavaughan
Copy link
Collaborator

Would have several applications, including assessment of catheter placement

@tavaughan tavaughan self-assigned this Oct 27, 2017
@tavaughan
Copy link
Collaborator Author

tavaughan commented Nov 1, 2017

Here is what ideally I'd like to have on the GUI two new parameters. An output node to store resampled points (either as Markups or as Model), and a resampling distance. See the two I added to the GUI (they're at the bottom of the list):

gui

Now I could easily change the low-level functions to have an optional output parameter (vtkPoints* resampledPoints) and resampling distance, but this will add even more parameters to these already-long interfaces:

    static void GeneratePiecewiseLinearCurveModel( vtkPoints* controlPoints, vtkPolyData* outputTubePolyData,
      double tubeRadius=vtkSlicerMarkupsToModelCurveGeneration::TUBE_RADIUS_DEFAULT,
      int tubeNumberOfSides=vtkSlicerMarkupsToModelCurveGeneration::TUBE_NUMBER_OF_SIDES_DEFAULT,
      int tubeSegmentsBetweenControlPoints=vtkSlicerMarkupsToModelCurveGeneration::TUBE_SEGMENTS_BETWEEN_CONTROL_POINTS_DEFAULT,
      bool tubeLoop=vtkSlicerMarkupsToModelCurveGeneration::TUBE_LOOP_DEFAULT,
      vtkPoints* outputResampledPoints=NULL,
      double resamplingDistance=RESAMPLING_DISTANCE_DEFUALT );

Obviously this will make things rather messy. What would be the best way to proceed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant