Skip to content

fix: housekeep tool and tests - #37

Merged
knicked merged 3 commits into
developmentfrom
kd/housekeep_fix
Jul 10, 2026
Merged

knicked merged 3 commits into
developmentfrom
kd/housekeep_fix

Conversation

@knicked

@knicked knicked commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Housekeep tool would ignore moveit errors and treat them as "side not available" (for housekeeping), making failing tests pass.

When a housekeep test fails (probably because of a moveit error, or a human getting deadlocked in the way of the robot), it makes no sense to run the next test, because it will also fail. So a simulation reset is triggered that will put the robot and human back into a valid configuration.

knicked added 3 commits July 10, 2026 13:42
Signed-off-by: Kacper Dąbrowski <kacper.dabrowski@robotec.ai>
Signed-off-by: Kacper Dąbrowski <kacper.dabrowski@robotec.ai>
Signed-off-by: Kacper Dąbrowski <kacper.dabrowski@robotec.ai>
@knicked
knicked force-pushed the kd/housekeep_fix branch from a2c0cb9 to 5ac0031 Compare July 10, 2026 11:42
@boczekbartek
boczekbartek self-requested a review July 10, 2026 14:32

@boczekbartek boczekbartek left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tested with cherry-picked: #35

git cherry-pick 97702f0076c9af8fa40356db09095ab3db514a12

LGTM!

@knicked
knicked merged commit 28025dc into development Jul 10, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants