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.

Sign up for GitHub

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

Jump to bottom

Update robot model to v4 #57

Merged
merged 2 commits into from
Apr 14, 2021
Merged

Update robot model to v4 #57

merged 2 commits into from
Apr 14, 2021

Conversation

goobta
Copy link
Member

@goobta goobta commented Apr 14, 2021

the new model fixes the axes issues and takes out the PDU. more details can be found in WPI-MMR/assets#7.

Can be auto merged

@goobta goobta added the new feature New feature or request label Apr 14, 2021
@goobta goobta enabled auto-merge April 14, 2021 14:48
@coveralls
Copy link

Pull Request Test Coverage Report for Build 748714342

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.05%) to 97.988%
Totals Coverage Status
Change from base Build 671632358: -0.05%
Covered Lines: 1315
Relevant Lines: 1342
💛 - Coveralls

andrew103
andrew103 approved these changes Apr 14, 2021
View reviewed changes
Copy link
Member

@andrew103 andrew103 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@goobta goobta merged commit 6a2a6c4 into master Apr 14, 2021
@goobta goobta deleted the goobta-v4-urdf branch April 14, 2021 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andrew103 andrew103 andrew103 approved these changes

@mahajanrevant mahajanrevant Awaiting requested review from mahajanrevant

Assignees

@goobta goobta

@andrew103 andrew103

@mahajanrevant mahajanrevant

Labels
new feature New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@goobta @coveralls @andrew103 @mahajanrevant