-
Notifications
You must be signed in to change notification settings - Fork 260
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
Add Redux Robotics links in third party libraries/devices #2256
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Edit: I'm bad and can't read.
Redux Robotics | ||
-------------- | ||
|
||
Redux Robotics currently offers the Canandcoder CAN-enabled magnetic encoder. The vendordep is necessary for CAN-based operation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The comment about vendordep seems a little odd because there's no link to the vendordep so it leaves the reader searching. It's also true of every other item on this page, but not stated. Recommend clarifying that it supports two modes of operation and library is only needed for CAN or removing, since that is covered in the linked manual.
321d5d4
to
aaa9888
Compare
the rebase is not right lemme try fixing this |
aaa9888
to
f1232d4
Compare
ok the changes are no longer merging in other stuff |
No description provided.