Skip to content
This repository has been archived by the owner on Jul 1, 2024. It is now read-only.

add orient command #52

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

add orient command #52

wants to merge 3 commits into from

Conversation

gabehowe
Copy link
Contributor

@gabehowe gabehowe commented Feb 7, 2024

This requires testing.
It operates on the same principle as this.

@OPstriker
Copy link
Member

There are a couple of issues with this,

  1. the Turn function is async, meaning that it runs in a different thread, and the controller sets the z rotation to 0 in the main thread, hence both conflicts.
  2. The function can't be run while driving, (would run poorly since it would conflict with the controller.)

@gabehowe
Copy link
Contributor Author

@OPstriker are you working on this or do you want me to fix it?

@OPstriker
Copy link
Member

@OPstriker are you working on this or do you want me to fix it?

I am currently working on this but could use some help, and I don't think this is critical atm

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants