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

Target speed ignored, rewrites all #22

Open
preussja opened this issue Apr 19, 2017 · 1 comment
Open

Target speed ignored, rewrites all #22

preussja opened this issue Apr 19, 2017 · 1 comment

Comments

@preussja
Copy link

preussja commented Apr 19, 2017

I noticed that my first layer speeds were being rewritten even thought they were different than the target speed. I then sliced a model in S3D, first layer 20mm/sec, bottom half 40 mm/sec and top half 60mm/sec. I set the target speed to 60mm/sec so that only the top half should have been painted. The whole gcode file ended up being painted. I am using the gui 0.5.4, though I think the issue was present earlier.

Archive.zip

@MarkWheadon
Copy link
Owner

The script is a little dumb at the moment -- whilst it ignores gcode lines with an explicit speed that's not the target speed, it doesn't ignore lines with no speed specified -- and lines with no speed specified use whatever speed was last set. So, you tend to end up with what some people are calling a welding line when the speed isn't the target speed (because the first vector has an explicit speed and so is ignored, but the rest have an implicit speed so they're processed).

There's a need for the script to keep track of the current speed -- when I've done that, we can decide whether stuff at other speeds remains unprocessed, or whether the script should just process everything (thus getting rid of the requirement for it to be sliced at a constant speed, which would make life simpler for everyone).

One reason I haven't made it just process all lines regardless of speed is I need to think of a way of reliably spotting retraction moves, as it wouldn't be right to change put a pattern into those!

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

No branches or pull requests

2 participants