search

aligator / GoSlice

This is an experimental slicer for 3d printing written in Go also usable as modular slicing lib.
Apache License 2.0
99 stars 16 forks source link

Slicing failed with error: error while calculating the union of new infill with already existing one #26

Closed tobychui closed 3 years ago

tobychui commented 3 years ago

Testing with the following file downloaded from the internet (ASCII stl) test.zip

which shows the following error when I am trying to slice it 2020-08-20_16-09-59

and I also noticed that the stl get overwritten by GoSlice when slicing failed, not sure if it is a bug or feature.

I tried to check the version of GoSlice but it seems neither do --version nor --v gives any versioning info. Maybe you can add in a command to check the version info as well?

Lastly, I love your work and find it amazing, add oil :)

-- Updates -- The information above is using the latest version that I clone from github this morning. Trying with the release v0.0.2 shows more error message as follows. Hope it helps. 2020-08-20_16-18-15

aligator commented 3 years ago

Thank you, I will take a look :-)

Also the version command is a good idea.

aligator commented 3 years ago

and I also noticed that the stl get overwritten by GoSlice when slicing failed, not sure if it is a bug or feature.

That is because till release 0.0.2 it always generated a test.stl file with the optimized stl for debugging. So if you do not name it test.stl it won't get overwritten.

This will be removed in future versions, I commented this out in the branch I am currently working on.

aligator commented 3 years ago

should be fixed with https://github.com/aligator/GoSlice/commit/8bd9432f1aeb3834d2ff6de4960682e38e5db72d also I disabled the test.stl output in the master. Could you please test it again?

tobychui commented 3 years ago

Nice, now I can slice the previous error file without any issues. Thank you so much for your hard work :))