bambulab / BambuStudio

PC Software for BambuLab and other 3D printers
GNU Affero General Public License v3.0
2.01k stars 273 forks source link

Imported STEP curves are not always converted to smooth curves in Bambu Studio (lack of precision) #1516

Open DzzD opened 1 year ago

DzzD commented 1 year ago

First, being able to use STEP files is great !! but...

When importing a STEP file, curves are converted to chained faces.

Sometime this conversion lack of precision (not enought faces), this result in face visible in the Bambu Lab studio but also on the printed part depending on the "Quality=>Precision=>Resolution setting" wich convert back chained faces to arcs when slicing.

A workaround is to increase "Quality=>Precision=>Resolution setting" a bit, but it would be better to improve STEP usage.

Two solutions : 1 - dont convert STEP curves to chained faces and use STEP curves for slicing (probably the hardest to implement). 2 - or when converting STEP in chained faces curves use more faces.

Below, a picture of an imported STEP file with two curves, the small one is ok, the big one show faces that will also appear in printed object :

CAD softwar (Fusion360), smooth curves : image

Bambu Studio, small curve ok, big curve show faces : image

Printed object, small curve ok, big curve show faces : image

Finally : With Fusion 360 exporting STL in high quality give better result than exporting STEP file which is too bad :(

zhimin-zeng-bambulab commented 1 year ago

@DzzD Import step file has a precision control, if the precision is too small, large file import will be very slow. The current precision can take into account the display effect and import speed. And it does not affect the print quality.

zhimin-zeng-bambulab commented 1 year ago

@DzzD Could you share your step file? I would like to use your file to test the print quality of the model again.

DzzD commented 1 year ago

Sure, I have attached it to this anwser.

Error on print only appear on the print if précision is set to default : image

Changing it to 0.02 hide theproblem, but this is more a workaround than a solution : image

The problem for me is that the import precision should be adaptative, higer on large curves and lower on small curves.

It should be expressed in mm and recursivly sub-divide generated faces until the original curve distance from generated faces get lower than the import precision parameter expressed in mm, so small curves (or flattened curves) will have a low number of faces, while big curves will have a lot more faces.

Flacon.step.zip

zhimin-zeng-bambulab commented 1 year ago

@DzzD I'm sorry that due to the software we are unable to implement it as you want. Because we use OCCT to process the mesh division that only provides linear deflection and angular deflection to control the division accuracy instead of absolute size.

DzzD commented 1 year ago

Well, dont be sorry ;) I understand that this would represent a lot of works as you are making the conversion in 3D, because eveything in the studio is made to work with 3D mesh composed of faces, wich imply a lot of restrictions.

It is like if you have made a bitmap image software and that I was asking you to work with vector images, not really feasible without a full redesign/re :)

The best of the best would have been to use the nurbs when slicing to get an exact interprétation of the STEP file curves but that would imply a full rework of the Studio, maybe in the futur you will have more times to handle this and will become also a leader in slicer ! like with your printer.

Anyway thanks for your time, keep up the good work !

zhimin-zeng-bambulab commented 1 year ago

Thank you very much for your understanding!

fiing commented 1 year ago

I don't know if it is 100% the same issue but I have problems too with round edges when imported from .step file. The item getting some "triangular" edges (1-2 mm in length) When I import the .step file in other programs the edges is as they should be without these triangular form.

billede

billede

zhimin-zeng-bambulab commented 1 year ago

@fiing Thanks! It is a bug for step file import, we will fix it.

zhimin-zeng-bambulab commented 1 year ago

related to #825

BambulabRobot commented 2 months ago

This issue has been marked as inactive due to no response for 90 days.