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

"Add Laser test fire function" #20452

Merged
merged 8 commits into from
Jan 10, 2021

Conversation

descipher
Copy link
Contributor

Requirements

LASER_FEATURE

Description

Add two menu items under the Laser Control SUBMENU when LASER_FEATURE is defined to fire a laser test pulse. Item 1 sets the adjustable fire time from 1 to 999 milli-seconds and the second menu item fires the trigger. A beep will occur if USE_BUZZER is defined. If PWM is supported it will fire at the power PWM setting that was previously set, if not set it defaults to 80%. No PWM results in full power with the set time duration.

Benefits

Supports alignment, testing and calibration of more powerful laser rigs.

Configurations

#define LASER_FEATURE
#define REPRAP_DISCOUNT_SMART_CONTROLLER

Related Issues

None

@LinFor
Copy link
Contributor

LinFor commented Dec 24, 2020

This feature could be very useful!

@descipher
Copy link
Contributor Author

Tested cleanup changes, looks good on the scope. Thanks :)

@shitcreek
Copy link
Contributor

Codes look about right to me, but I won't be able to flash and test it until Thursday.

Copy link
Contributor

@shitcreek shitcreek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Flashed and tested working.

@thinkyhead thinkyhead merged commit cf1f8af into MarlinFirmware:bugfix-2.0.x Jan 10, 2021
@descipher descipher deleted the Add.Function.Laser branch January 20, 2021 20:17
kpishere pushed a commit to kpishere/Marlin that referenced this pull request Feb 19, 2021
zillarob pushed a commit to zillarob/Marlin that referenced this pull request Feb 25, 2021
W4tel-BiDi pushed a commit to W4tel-BiDi/Marlin that referenced this pull request Apr 5, 2021
thinkyhead pushed a commit to thinkyhead/Marlin that referenced this pull request Apr 29, 2021
thinkyhead pushed a commit that referenced this pull request Apr 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants