Skip to content
This repository has been archived by the owner on Dec 1, 2020. It is now read-only.

WIP: Fix loading home gait error #108

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

BasVolkers
Copy link
Contributor

Closes PM-219

Description

Fixes error with loading home gait

Changes

Added check that every joint does not have one setpoint

Testing

@codecov
Copy link

codecov bot commented Aug 27, 2020

Codecov Report

Merging #108 into develop will decrease coverage by 0.11%.
The diff coverage is 54.54%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #108      +/-   ##
===========================================
- Coverage    72.33%   72.21%   -0.12%     
===========================================
  Files           17       17              
  Lines         1623     1634      +11     
  Branches       176      177       +1     
===========================================
+ Hits          1174     1180       +6     
- Misses         404      408       +4     
- Partials        45       46       +1     
Flag Coverage Δ
#production 54.36% <0.00%> (-0.23%) ⬇️
#test 98.48% <85.71%> (-0.14%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...ch_rqt_gait_generator/gait_generator_controller.py 83.62% <0.00%> (-1.21%) ⬇️
...r/test/nosetests/gait_generator_controller_test.py 98.61% <85.71%> (-0.26%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8cfe7a7...bbe46c9. Read the comment docs.

@Olavhaasie Olavhaasie marked this pull request as draft August 28, 2020 13:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant