-
Notifications
You must be signed in to change notification settings - Fork 2
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
Fast Foward Gym Solo #55
Conversation
Pull Request Test Coverage Report for Build 649393226
💛 - Coveralls |
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.
Just a couple of explainatory comments. As always, lmk if there are any questions on anything :)
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.
I just have a question but its minor so approving it
|
||
while True: | ||
pos = float(input('Which position do you want to set all the joints to?: ')) | ||
if pos == 69.: |
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.
What is up with this xD?
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.
oh that's the value to reset the robot lol. I'll open up another PR and get rid of it lmfao
Why do we have solo8_mmr now? |
We realistically only have 2 versions: ODRI's Solo and ours. Since we aren't really tracking revisions, I figured it's probably easier just to make a unified name rather than |
Kinda of a huge PR, sorry for the bulk. tl;dr:
gym_solo/assets
is now a submodule to https://github.com/WPI-MMR/assets. We can use that repo for issue tracking and everything will get updated here as wellI'll also add comments in the PR as well to help explain some of my logic and reasoning.