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

Bez3 integration tests #526

Open
wants to merge 43 commits into
base: master
Choose a base branch
from
Open

Bez3 integration tests #526

wants to merge 43 commits into from

Conversation

lkunelk
Copy link
Contributor

@lkunelk lkunelk commented Jun 26, 2022

  • Only the minimal change is included in this PR the commits are kept minimal
  • Code is readable and documented in the code
  • Minimal tests have been added

@lkunelk
Copy link
Contributor Author

lkunelk commented Jun 26, 2022

  • robot is able to walk in webots
  • can't get up, but with longer hands it was able to
  • still need to add pressure sensors to the feet

Vuwij
Vuwij reviewed Jun 26, 2022
View reviewed changes
.gitmodules Outdated Show resolved Hide resolved
@Vuwij
Copy link
Member

Vuwij commented Jun 26, 2022

Integration CI is not passing, possible due to some leftover settings

lkunelk and others added 8 commits June 26, 2022 12:34
@lkunelk
remove urdf2webots submodule
09aaa8c
@lkunelk
temp
e57de81
@lkunelk
make bez3 walk in pybullet
3dd2092
@lkunelk
firmware
7215b3e
@Vuwij
Seperate bez13 int test (#532)
06995c3 
* Update files here

* Load docker images

* Update test_integration.yml

* Tag and pull change

Co-authored-by: Jason Wang <vuwij1@gmail.com>
@lkunelk
fix byte string problem
1c8219b
@lkunelk
undo display/ros for test
25f936d
@lkunelk
wip
a2c6986
@lgtm-com
Copy link

lgtm-com bot commented Jun 30, 2022

This pull request introduces 1 alert when merging a2c6986 into 828e015 - view on LGTM.com

new alerts:

  • 1 for Unused import

@lkunelk
fix imu issue
4c5982b 
- imu link name was wrong in xacro -> messes with tf tree stuff
- fixed initial height that caused sudden jump
@lkunelk
remove unused script
fb402b3
@lgtm-com
Copy link

lgtm-com bot commented Jul 1, 2022

This pull request introduces 1 alert when merging fb402b3 into 828e015 - view on LGTM.com

new alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Jul 1, 2022

This pull request introduces 1 alert when merging 335611a into 828e015 - view on LGTM.com

new alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Jul 1, 2022

This pull request introduces 1 alert when merging 1204507 into 828e015 - view on LGTM.com

new alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Jul 1, 2022

This pull request introduces 1 alert when merging 31d88f0 into 828e015 - view on LGTM.com

new alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Jul 1, 2022

This pull request introduces 1 alert when merging 40fe92e into 828e015 - view on LGTM.com

new alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Jul 2, 2022

This pull request introduces 1 alert when merging 9e91472 into 52b6ecd - view on LGTM.com

new alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Jul 2, 2022

This pull request introduces 1 alert when merging 2c78ab9 into a6480ee - view on LGTM.com

new alerts:

  • 1 for Unused import

@Vuwij Vuwij changed the title Bez3 proto Bez3 integration tests Jul 4, 2022
Vuwij and others added 16 commits July 4, 2022 17:09
@Vuwij
Reset
dc46aa9
@lkunelk
Merge remote-tracking branch 'origin/master' into bez3-proto
e1c9233
@lkunelk
use updated arm length
6551033
@lkunelk
Merge remote-tracking branch 'origin/master' into bez3-proto
f7b7b70
@lkunelk
bez walks slighlty faster with stable camera
ab4a3ae
@lkunelk
walk faster, problem with initial step making robot turn
f11bfc6
@lkunelk
add bez3 trajectories
4ceca60
@lkunelk
get branch up to date with master
e4b3d44
@lkunelk
Merge remote-tracking branch 'origin/master' into bez3-proto
4fe976d
@lkunelk
Merge branch 'master' into bez3-proto
7e7067c
@lkunelk
Merge branch 'master' into bez3-proto
3cfd1d5
@lkunelk
zero standing PID and make walking slower but more stable
b31170f 
- I think the PID was causing the robot to fall initially
@lkunelk
bez3 stands higher while walking
a476e65
@acrylic-origami
Various fixes for Bez 3 get up from back testing
6331f48
@acrylic-origami
Elongate arms of Bez 3 by 10 cm
17c76f8
@acrylic-origami
Merge branch 'bez3-proto' of github.com:utra-robosoccer/soccerbot int…
9c92b65 
…o bez3-proto
@github-actions
Copy link

github-actions bot commented Sep 9, 2022

Stale pull request message

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Vuwij Vuwij Vuwij left review comments

@acrylic-origami acrylic-origami Awaiting requested review from acrylic-origami

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@lkunelk @Vuwij @acrylic-origami