Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adds always-on ramping into area cut #181

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dognotdog
Copy link

I added ramping into a cut, seems to produce the right gcode, but should of course be reviewed and the option from the GUI to do ramping is also currently ignored, but that shouldn't be very hard to add, except that I don't know the HeeksCNC internals very well.

The algorithm basically tries a 45deg or shallower ramp, but vertex-by-vertex, so sometimes the ramp can be very shallow if the vertices are far apart, but that shouldn't be a problem. The ramp ends at the curve's starting point, going backwards, so cutting direction is kept.

Had to do this as otherwise my mill isn't far from shaking itself apart when using crappy end mills for pocketing operations.

Cheers!

@neomilium
Copy link
Member

How do you use it if not with GUI ?

@dognotdog
Copy link
Author

I've just hardcoded it to be always-on, as for the project I used it for that was a decent way to do things, but obviously thats a dirty hack :)

@bullestock
Copy link

I have created a fork at https://github.com/bullestock/heekscnc/ which, among other fixes, includes the GUI to select Entry Move.

@neomilium
Copy link
Member

@bullestock I can integrate your changes if you make them available through small (ie. easy to read and review) Pull-Requests.

Thanks!

@bullestock
Copy link

I can integrate your changes if you make them available through small (ie. easy to read and review) Pull-Requests.

@neomilium Alas, I don't really have neither the time nor the inclination to do that - it was mostly a quick hack to get stuff working for my own use.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants