zhangty019 / S3_DeformFDM

S^3-Slicer: A General Slicing Framework for Multi-Axis 3D Printing
BSD 3-Clause "New" or "Revised" License
87 stars 14 forks source link

New TetMesh model: selectionFilter file #9

Closed yvanblanchard closed 1 year ago

yvanblanchard commented 1 year ago

Hello,

I'm trying to use my own TetMesh model in your framework. I modified the "updateFramewokParameter" method for my model name, but it now tried to load and read a corresponding "selectionFilter" text file. The existing selectionFilter files contain chinese characters, so I can't try to understand which kind of data I have to define for my model (for creating my own selectionFilter).

Could you please help me with that ? Thank you

yvanblanchard commented 1 year ago

I noticed that the content of those selectionFilter text file is not in Chinese when previewing them directly from GitHub.

yvanblanchard commented 1 year ago

@zhangty019 I tried to copy one of this selectionFilter file, renaming it with the name of my tetMesh file, for running the slicer, but processing hangs... Could you please try by yourself, and tell me how to proceed (if succeed at your side) ? I share my TetMesh file (thing.tet). Thank you thing-tetMesh.zip

yvanblanchard commented 1 year ago

@zhangty019 Could you please help me on that point ? I'm stucked on this for the moment..

zhangty019 commented 1 year ago

Hi @yvanblanchard, please check the attachment for details. Issue 9.zip

yvanblanchard commented 1 year ago

Thank you @zhangty019 I will try that now.

Just some quick questions before:

Sorry for the questions, but it's for better understanding at my side : Thank you again for your help.

zhangty019 commented 1 year ago

Please check if these are helpful for you.

  1. It is not necessary to select the bottom, the other region can also be selected, it is just an example.
  2. My PC is of 32GB RAM, when the size is more than 32GB, windows will borrow the visual memory from the hard disk. The direct method to decrease memory usage is to use the file with fewer tet elements.
  3. Yes, the 8MB limit is from memory. What I did is to first get the boundary of the model and then do sparse re-mesh and generate the tet model again.
yvanblanchard commented 1 year ago

@zhangty019 I tried following your requirements, and it works much better now, thanks. But new remarks:

yvanblanchard commented 1 year ago

I forgot to ask: you mentioned that the coordinate system (of input mesh model) was "Y up", and then modified (transformed to be "Z up" ?), but it seems my mesh (stl/obj, and then tet mesh) was 'Z up', in the expected printing direction (see image, blue line of coord sys is Z): image

So why the need to modify/reorient it ?

Could you also please share with me the new outer mesh (coarsy), that you used for the tet filler (and generate the new 'thing_updated.tet' file)?

Thank you again,

zhangty019 commented 1 year ago

@zhangty019 I tried following your requirements, and it works much better now, thanks. But new remarks:

  • what is the exact purpose of this selection file ? (especially if it's not important to select the base area or the other mesh part)
  • when clicking on toolpath button (5), the Bunny model was inserted and displayed in viewer ! image
  • I got errors when computing the toolpath (in console window:) image
  • I got a crash (exception) when computing the 'Fabrication' (last tab-page)
  1. In the SQ (surface quality kept) case and other hybrid cases which include the SQ requirement, the selection file is needed to let the code know which part of the model surface should be kept.
  2. Please refer to Step 4 of Usage in README, the layers should be remeshed before toolpath generation and they are saved into /DataSet/remesh_operation/layers_remeshed/
  3. The warning means, in the small patch, there is not enough space to generate zigzag toolpaths after the 3-ring boundary. It is ok, some protection is done already.
zhangty019 commented 1 year ago

I forgot to ask: you mentioned that the coordinate system (of input mesh model) was "Y up", and then modified (transformed to be "Z up" ?), but it seems my mesh (stl/obj, and then tet mesh) was 'Z up', in the expected printing direction (see image, blue line of coord sys is Z): image

So why the need to modify/reorient it ?

Could you also please share with me the new outer mesh (coarsy), that you used for the tet filler (and generate the new 'thing_updated.tet' file)?

Thank you again,

The deformation and bottom detection algorithm is using the Yup rules, and I transform them into Zup for the fabrication steps.

yvanblanchard commented 1 year ago

Ok , it's clear now. thanks a lot again

zhangty019 commented 1 year ago

thing_updated.zip And also this zip is for the sparsed model (.obj)