Skip to content

Write skeleton as a vtkPolyData#1

Open
myousefi2016 wants to merge 1 commit intor03ert0:masterfrom
myousefi2016:master
Open

Write skeleton as a vtkPolyData#1
myousefi2016 wants to merge 1 commit intor03ert0:masterfrom
myousefi2016:master

Conversation

@myousefi2016
Copy link

No description provided.

@r03ert0
Copy link
Owner

r03ert0 commented Jan 22, 2019

Hi @myousefi2016,
thank you for your pull request!
(I haven't tried it yet, though)

Does it require to install VTK? In the past installing VTK has been a bit tricky. I wrote this vtk reader/writer that doesn't require the complete VTK to work: https://github.com/r03ert0/vtk_mesh_io
Maybe it could be used instead?

Also, it seems that the VTK export replaces the cgal export. Would it be possible to have it as an alternative? I have code that uses skeleton and it will break if there's no cgal output anymore...

Thanks again!

@myousefi2016
Copy link
Author

Hi @r03ert0,
Thanks for comments.

  • Does it require to install VTK?

Yes it needs to install VTK Python bindings to work but it could be installed easily by using these commands: pip install --upgrade --user pip && python -m pip install --upgrade --user vtk or if you have Anaconda2 or MiniConda: conda install vtk.

No we can't use your vtk mesh io code cause as you can see in the code we need to have access to other vtk functionalities to store cells, points, lines, etc. So, I think it's better to stick to official vtk python release.

  • Also, it seems that the VTK export replaces the cgal export. Would it be possible to have it as an alternative? I have code that uses skeleton and it will break if there's no cgal output anymore...

Please give me your desired cgal output and I will add it to the new script. Cause, in your original python script, it just read the *.cgal files produced by cgal and just print the vertices and edges and it doesn't convert them to something useful in order to be able to render them easily.

I'm waiting for your opinion. Thank you!

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.

2 participants