search

SoMa-Project / ec_grasp_planner

Grasp Planner Based on Environmental Constraint Exploitation
BSD 3-Clause "New" or "Revised" License
7 stars 3 forks source link

Model separation grasp success estimation (apply fix to ocado branch) #52

Closed JannisW closed 5 years ago

JannisW commented 5 years ago

The most recent version of the grasp success estimator had a bug that broke the difference mass model in case the robot noise was too big. The difference model was clearly separated from the absolute model and the problem was fixed in c5fc040483afc648f3c6456985cfdec0d4cc6944 and https://github.com/SoMa-Project/soma_utils/commit/759068041dfe96d4f18515827f82bcca5f55dc8f

This is merged into tub_ocado_merge and should be propagated to your branches as well (This should not require any manual merging in case of a git cherry-pick of the commits...)

ptriantd-ocado commented 5 years ago

Thanks! We have propagated the changes to our branches...