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

Option to use stepper motor as a winder #16

Open
wants to merge 5 commits into
base: Release-Candidate
Choose a base branch
from

Conversation

Mylan719
Copy link

This pullrequest implements an option to use X axis stepper motor as a winder instead of Pin 8 DC 12V motor by uncomenting #define USE_STEPPER_WINDER in the configuration.h file. Pin 8 then works as ordinary cooling fan.
This implements feature described in #15.

When working on the feature I found 2 bugs that i fixed:

  • When motion menu was used to move an axis, the planner used old current position of X or Y or Z axes to plan unwanted move when extrusion was resumed. The issue was fixed by clearing the stepper positions besore resuming the extrusion.

  • When DC motor winder on Pin 8 was used the winder did not stop when extrusion was paused. Thus the winder became unstoppable. I fixed the issue by setting winderSpeed to 0 and in the main loop the speed is set to winderSpeed only if extrude_state is enabled.

Mylan719 added 5 commits June 11, 2022 14:26
The winder is not started/stopped if USE_WINDER_STEPPER is defined
Bug fixed where winder was not stopped on "Stop extruder"
Renamed the winder pin to make it clear it can be used a fan
- Only reset the winder fan speed in main loop if the extrude_status
if the extrude_status is enabled. Otherwise it keeps resetting the
speed even if set to a value by Gcode or LCD.
- This is to fix a bug where first move after resume was planed with old
value of current_position.
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.

1 participant