-
-
Notifications
You must be signed in to change notification settings - Fork 361
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
feat(objwriter): add support for vtkOBJWriter #2405
base: master
Are you sure you want to change the base?
Conversation
68353a6
to
175a919
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/** | ||
* vtkOBJWriter writes wavefront obj (.obj) files in ASCII form. OBJ files | ||
* contain the geometry including lines, triangles and polygons. Normals and | ||
* texture coordinates on points are also written if they exist. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you might want to mention that materials are not written out.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have written the code for the mlt part, but i don't know how it can be exposed? user need to pass the vtkTexture? or just pass the texture filename?
@jourdain do you have any idea how to expose the texture export?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, a vtkTexture (or vtkImageData?) would need to be optionnally passed (i.e. as a second input macro.algo(publicAPI, model, 2, 1);
)
I guess it could output the mtl part in a second output (i.e. macro.algo(publicAPI, model, 1, 2);
and outData[1] = ...
)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok i will re-add the mtl code again
fix #2404
Context
Add support for vtkOBJWriter
Results
Changes
PR and Code Checklist
npm run reformat
to have correctly formatted codeTesting