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

Safe pattern transition in StrokeEngine mode #138

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Ptibibi
Copy link

@Ptibibi Ptibibi commented Nov 4, 2024

Safe pattern transition in StrokeEngine mode

Software/src/ossm/OSSM.Preflight.cpp

  • Fix preflight to waiting that stepper is at home position
  • Available for strokeEngine.transition state

Software/src/ossm/OSSM.StrokeEngine.cpp

  • Motion steppper only in idle state
  • Stepper position is already initialized by homing function

Software/src/ossm/OSSM.h

  • Create strokeEngine.transition state to set preflight task before idle state

@Ptibibi Ptibibi changed the title Ptibibi/stroke engine safe transition Safe pattern transition in StrokeEngine mode Nov 4, 2024
@Ptibibi Ptibibi marked this pull request as draft November 4, 2024 17:03
@Ptibibi Ptibibi force-pushed the ptibibi/StrokeEngine_safe_transition branch from 4fa554a to 1050890 Compare November 4, 2024 17:42
@Ptibibi Ptibibi marked this pull request as ready for review November 4, 2024 17:44
task available for strokeEngine.transition state
if speed is set at 0% before strepper is at home position
task is not finished correctly and stepper is not in correct step

Signed-off-by: benoit bobert <[email protected]>
Create strokeEngine.transition state
Set Preflight before return in idle (safe stepper position)

Signed-off-by: benoit bobert <[email protected]>
add strokeEngine.transition state in available state
motion stepper only in idle state
steeper init position is set by homing function

Signed-off-by: benoit bobert <[email protected]>
@Ptibibi Ptibibi force-pushed the ptibibi/StrokeEngine_safe_transition branch from 1050890 to 1d88038 Compare November 6, 2024 22:58
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