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

✨[FEATURE] - More Tracking Wheel Support #165

Open
ssejrog opened this issue Aug 27, 2024 · 1 comment
Open

✨[FEATURE] - More Tracking Wheel Support #165

ssejrog opened this issue Aug 27, 2024 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@ssejrog
Copy link
Member

ssejrog commented Aug 27, 2024

Is this feature related to a problem?

EZ-Template only supports 0 tracking wheels or 2 tracking wheels.

What should be added?

Teams have been using an IMU+1 tracking wheel for odometry. EZ-Template should support this, as well as one parallel tracker and one perpendicular tracker.

OPTIONAL: Describe any solutions you've considered

OPTIONAL: Additional context

@ssejrog ssejrog added the enhancement New feature or request label Aug 27, 2024
@ssejrog ssejrog self-assigned this Aug 27, 2024
@ssejrog
Copy link
Member Author

ssejrog commented Aug 30, 2024

This needs to account for having a single parallel tracker in normal drive motions. This tracker should only be used for odom motions and should not be used for driving motions, as there is independent PID for the left and right sides of the drive. EZ-Template can already handle two parallel trackers.

ssejrog added a commit that referenced this issue Sep 5, 2024
With this comes a new constructor that is unique to not overlap with "ratio" used previously, and removing "ratio" will create breaking changes.  This new constructor takes in a motor group and it extracts that out into the ez motor groups.  This currently doesn't work as there's a bug with `get_direction` that has an open issue on the PROS kernel.

If a left and right tracker are enabled, the main sensors for the drive switch over to being these tracking wheels.  If only one sensor is used, then this tracker is only used for odom.  Normal drive motions will use IMEs still, as that PID uses a left and a right PID.

Only one horizontal tracker will be used.

Angle is currently always calculated with IMU.  Until there's an easy way to tune track width, #45 will remain open.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant