remove old type, when new type assigning.

tue-robotics / ed_perception

ED extension for classifying entities based on their attached RGBD measurements.

2 stars 5 forks source link

remove old type, when new type assigning. #28

Closed MatthijsBurgh closed 6 years ago

MatthijsBurgh commented 6 years ago

fixes #25. First time inspecting the cabinet in simulation

rosservice call /amigo/ed/simple_query "id: '94a87f4c8c6e45ce1f813ae510713fed'" 
entities: 
  - 
    id: 94a87f4c8c6e45ce1f813ae510713fed
    type: hairspray
    types: ['hairspray']

Inspect cabinet again. Random classifier returns other label.

rosservice call /amigo/ed/simple_query "id: '94a87f4c8c6e45ce1f813ae510713fed'" 
entities: 
  - 
    id: 94a87f4c8c6e45ce1f813ae510713fed
    type: ice_tea
    types: ['ice_tea']

MatthijsBurgh commented 6 years ago

merge together with tue-robotics/ed#57

rokusottervanger commented 6 years ago

Looks good, but I don't know how to verify

MatthijsBurgh commented 6 years ago

travis fails, because the correct branch of ED is not checked out.

MatthijsBurgh commented 6 years ago

Tested on AMIGO

rokusottervanger commented 6 years ago

I'm just going to trust you tested this thoroughly