Skip to content

RFC0017 switching between FOL and manual mode#15

Merged
Ahelsamahy merged 4 commits intoRFC0008_Followingfrom
RFC0017_SwitchingBetweenFOLandManualMode
Feb 28, 2024
Merged

RFC0017 switching between FOL and manual mode#15
Ahelsamahy merged 4 commits intoRFC0008_Followingfrom
RFC0017_SwitchingBetweenFOLandManualMode

Conversation

@Ahelsamahy
Copy link
Collaborator

The Following (FOL) mode keeps working even if the user is touching the active area on the mobile controller. This raises the problem if the user wants to change the immediate action of the robot in case of an accident. This feature will allow the user to switch from Following mode to normal mode as fast as possible.

What is changed:

  • Wrap sending and changing the states of FOL button in class
  • Implement the switch to manual mode from following if the user touches one of the two triangles (of active area)

As the old function could be only called on the button call and couldn't be exported. I had to wrap it in a function I can export, but the function was doing more than one process (changing the text on button and sending POST with the command) so I split it into class.
…or the button

There was implementation done for hiding the button on immediate break (part of the smooth breaking RFC16) and now it needs another way to access it through the class
In hiding and showing the button while the user is interacting with the active area
It will read the current state of button before it starts drawing the triangles where the OP fingers are placed
@Ahelsamahy Ahelsamahy self-assigned this Feb 28, 2024
Copy link
Collaborator

@OrestisTS OrestisTS left a comment

Choose a reason for hiding this comment

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

dayum

@Ahelsamahy Ahelsamahy merged commit 7158fd4 into RFC0008_Following Feb 28, 2024
@Ahelsamahy Ahelsamahy deleted the RFC0017_SwitchingBetweenFOLandManualMode branch February 28, 2024 09:33
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.

2 participants