Closed mir7b closed 2 years ago
Done.
I can not reach into the collision meshes with giskard. Bullet needs convex decompositions for the collision meshes: https://github.com/bulletphysics/bullet3/issues/1507 Can you please change that?
an alternative would be to give the collision entries multiple box geometries, similar to how it is done for the tiago: https://github.com/pal-robotics/tiago_robot/blob/ee8c909be59aec65cf707c02013cfed37be2d051/tiago_description/urdf/torso/torso.urdf.xacro#L116
- This is debatable, but I think the joints should all open when the joint state increases
Yes, I require this as well.
Could you also make the handle coordinate frames as defined here: https://ai.uni-bremen.de/wiki/3dmodeling/items
Meh, convex hull of the apartment walls is the whole apartment because it's one mesh x_x. So everything is in a collision. Looking forward to @ichumuh's obj meshes.
Otherwise everything else looks good to me.
Final todos:
Ah, yes. The difficult part. To describe a location inside the top left cupboard I need a link for the cupboard, or at least the bottom board of cabinet1 top part needs to be its own link. Otherwise I cannot explain to the robot where the cup is using the URDF links that already exist. Is it possible to cut off parts from a mesh and separate it into two meshes? With FreeCAD or something? I guess I'll request this in a separate PR. We will need to model the dishwasher drawers anyway. So the cupboard board link and the dishwasher drawers can be in this separate PR.
Should be okay to check out now.
I noticed links like "sa" "sb" "side1" etc., for sockets I guess. Would be good if they had more explicit names.
considering those have no functionality, I will leave the proper naming of them to the final tweaks. Side A and side B are the two sides of the kitchen, I insist on that name staying because they make life easier in tweaking.
This is a bit unrelated, but it would be cool if you centered the apartment such that the global map origin is in the middle of the room.
Check out after this commit. this should be fixed now.
There is still kitchen2_distplay.launch that has Simon's comments on it. Either remove the launch file or address the comments.
Check the new commit.
more-or-less ok position for the kitchen in the apartment,
it's open for tweak. one just needs to know their way around a URDF to tweak it.
Could you also make the handle coordinate frames as defined here:
done.
To describe a location inside the top left cupboard
I could have done that easily at the beginning by changing their centre in blender, and then putting them in the kitchen URDF. But now this will cause a lot of rework. Let's discuss this later and find a solution.
I think the joints should all open when the joint state increases
fixed now. you might want to change the wardrobe that's already in there though.
display.launch file
already removed and merged as xacro in the apartment.
is there anything else to be addressed?
Ooops! Accidentally hit "close" instead of "comment"!
This is a bit unrelated, but it would be cool if you centered the apartment such that the global map origin is in the middle of the room. @Sanic should we change the origin in the Unreal mesh or rather in the URDF? The mesh has a weird origin, doesn't it?
Having this consistent would be great! I can send @mir7b the Link to the UE4 apartment. I think that the origin in the UE4 model should be close to the center of the room.
Can we make it in a corner? Like the left left corner near the kitchen? It is like that in the refills lab, makes physically measuring stuff in the real world easier.
The origin is below ground. Aka the whole furniture is floating in the air.
The origin should be directly on the floor level, or?
what about the tf static transform publisher and the joint state publisher in the bringup.launch?
The bringup.launch from Giang?
Can you add what you need in the obj PR? I have my own launch file, so I don't care.
what about the tf static transform publisher and the joint state publisher in the bringup.launch?
doesn't need it. you can use it the same way you use the apartment. It is merged in there anyways.
Awesome! I have one request before testing this. Could you please merge your kitchen2 directory with iai_apartment? You could simply include your kitchen.urdf in the apartment.urdf, that would be nice and modular. That would require putting all the meshes of kitchen2 into a subdirectory of apartment. That way, the meshes would also not have to be called kitchen2 but iai_apartment/kitchen, which makes it clearer why there are two kitchens. The idea of using this URDF is that we would spawn the robot always in the complete apartment with walls, couches and the kitchen, instead of only using the kitchen.