Stage 1C incorporation into educator's guide and several overview pages - #251
Stage 1C incorporation into educator's guide and several overview pages#251Venture-Capitalist wants to merge 23 commits into
Conversation
….mdx Co-authored-by: Zach Harel <zach.harel42@gmail.com>
….mdx Co-authored-by: Zach Harel <zach.harel42@gmail.com>
…nresolved and need to be sorted out before merging
….mdx Co-authored-by: Adriana <16786568+Adrianamm@users.noreply.github.com>
….mdx sounds good boss Co-authored-by: Adriana <16786568+Adrianamm@users.noreply.github.com>
github-actions
Bot
added
Curriculum material
|
🌐 Preview URL: https://pr-251-frcsoftware.frcsoftware.workers.dev |
There was a problem hiding this comment.
One question - I originally envisioned Stage 1C as being more about having drive forward + turn commands using PID, but if we want to build 1C around aiming at the hub or a target that also works, probably seems more immediately applicable to FRC anyways. Are you envisioning this as turning to a single fixed angle or more of the sort of dynamic aiming towards the hub from anywhere on the field like you would with actual vision? My only concern with that approach is that it would require actual Pose info, and while we technically have that through sim, user code isn't running any odometry.
There was a problem hiding this comment.
The way I understood the plan earlier, the idea was to have the user code a command which enables the tank drive to rotate towards a field-relative heading angle passed in by the user (potentially through telemetry if we wanna do that). The project doesn't necessarily have to be a full hub autoaim system; that's a lot more complex imo. On the translational side of things, I'm perfectly good with having the tank drive have forwards/backwards PID commands too if that's what we want.
I think in all cases we'd need to also have some odometry info because you can't really run any feedback loop without encoder data. You can probably get away with just the tank chassis odometry, but this wouldn't necessarily be an atrocious time to introduce users to reading IMU data either.
|
Just checking, this is for #252 right? |
4 participants
Description
What changed?
Why? (the code + comments should speak for itself on the "how")
Added 1C to the educator's guide via a blank WIP page. Also added 1C mentions to overview pages in both the learner's course and educator's guide.
Closes #159
Any issues this pull request closes or pull requests this supersedes should be linked with
Closes #issuenumber.Meta
Merge checklist: