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

Started on wristevator setSetPoint command #45

Closed
wants to merge 3 commits into from

Conversation

reesesbowls
Copy link
Collaborator

What I was trying to do (lowk dont know how to do trapezoid profiles so ignore last lines):
image

@PGgit08
Copy link
Contributor

PGgit08 commented Jan 25, 2025

I'll post a question on cd today.

@elvizer
Copy link
Member

elvizer commented Jan 25, 2025

This method has some issues:

  • To make the elevator and wrist reach the setpoint taking the same amount of time, you can modify their CruiseVelocity and Acceleration config on the fly. However, calculating for this is difficult because the robot moves fast and doesn't have much time to reach the cruise velocity.

This method could work if its reliable to calculate the cruise velocity and acceleration to reach that setpoint within that time frame.

@PGgit08
Copy link
Contributor

PGgit08 commented Jan 27, 2025

cd post

@PGgit08 PGgit08 closed this Jan 28, 2025
@PGgit08 PGgit08 deleted the Wristevator-setSetpoint branch January 28, 2025 23:16
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.

3 participants