Skip to content
New issue

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

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

Already on GitHub? # to your account

Default pulse duration is actually 60ms (milliseconds) - not 60us (microseconds)(?) #6

Open
ThePhilLee opened this issue Oct 27, 2015 · 2 comments

Comments

@ThePhilLee
Copy link

ppmValue = "L60000 P%.2f B1 D0" % altppm

This seems to be the line that assigns the pulse duration. Assuming the units for "L60000" is microseconds (10^-6), the default pulse duration is actually 60 milliseconds (10^-3). The readme says 60us, while the plugin help tab says 60ms.

I burned some lines at feed=5000 mm/min ppm=0.075 and default L60000 as a test to see how long the actual pulses were. It resulted in dashed lines, and each dash has a width of 4.6mm-5.2mm. Unless my math is wrong, at 5000 mm/min, a pulse of 60 milliseconds would result in a line width of 5 mm. A pulse of 60 microseconds, on the other hand, would result in a width of .005 mm.

On a related note, it would be great if we could assign the pulse duration in each layer as we do the feed and ppm. Or at the very least, if we can assign a default pulse duration. Great plugin by the way.

@TurnkeyTyranny
Copy link
Owner

Hey nice find! I think you're right, the units are probably typo'd. I'll check this out.

@dreydean
Copy link

Any news on this?

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

No branches or pull requests

3 participants