fix cut inside inside inside (copied from file) about bark-beetle-parametric-toolpaths HOT 7 CLOSED

I think I solved this for both cutout and pocket. I copied a piece of code and did some fixing on making the trees match. I've only tested on the example below (gray uses old code, green is using edited code). In the pocket component this also seems to solve some unwanted artifacts happening in this specific case (see red marked areas).

I think I'll first do a bit more testing before replacing the pocket and cutout component and committing to Github. @JensDyvik Do you happen to have some kind of file with a lot of different geometry to test on?

from bark-beetle-parametric-toolpaths.

I started doing some testing and found out I also had to do a fix in the case of using only 1 curve. Now it seems to be working quite well. In the case of the pocket it seems like it's also way faster.

from bark-beetle-parametric-toolpaths.

Tried committing with Github. But I get an error saying "Failed to sync this branch - You might need to open a shell and debug the state of the repo."

from bark-beetle-parametric-toolpaths.

Seems like it worked now. I completely disabled my antivirus.

from bark-beetle-parametric-toolpaths.

Very exciting to have this fixed. Thank you very much. The fix has been included in the main CNC milling example in the latest development folder: b1dd65f

I dont have a specific torture testing docuement. Maybe you could upload the one you made to a test object folder in the repo?

from bark-beetle-parametric-toolpaths.

By the way, as mentioned before, the planar surface function is a potential method for solving endless objects inside objects. And extracting isocurves from the planar surfaces could potentially be a fast and reliable way of creating pocket raster lines.

from bark-beetle-parametric-toolpaths.

I added a folder with the curves I've been using for testing. Some caused issues before and are still causing issues (the H with the snake right now for example). I'll also keep using this as a place to store new objects that cause issues while I encounter them.

Yeah the planar surface method for solving endless objects inside objects is a really good idea. It requires a bit more data tree matching than the solution I integrated now. I think for now this works but it's definitely something to look into for the future!

from bark-beetle-parametric-toolpaths.