search

PadLex / SvgToGcode

MIT License
46 stars 27 forks source link

movement height and cutting height added #7

Closed tommy3001 closed 3 years ago

tommy3001 commented 3 years ago

I want to add 2 parameters: movement_height: Traveling height above object. cutting_height: height during laser, or cutting / engraving / drilling.. These both parameters makes the module much more usable for other applications, for example drilling.

PadLex commented 3 years ago

Having an optional movement height makes sense to me. What's the point of defining a cutting height though? The cutting height should decrease with every pass by pass depth. Maybe we should allow users to define a starting height (material height + focus/tool length).

PadLex commented 3 years ago

Currently the library uses relative movement for the z axis so this might not be so simple to implement. Since this library is targeted towards laser-cutters (I should make that clearer) I don't see a point to adding a movement height. Unfortunately CNC machining is a lot more complicated and I don't think this library will evolve in that direction.

Even though I'm not accepting this pull request, I'd like to thank you for taking your time to make it. Do let me know if I misunderstood anything or you disagree with my thought process,