Add Grasping Deep Learning post#512
Conversation
|
Looks good. We just need to remember to update the link to the tutorial to the official moveit_tutorials repo once moveit/moveit_tutorials#521 is merged. I just approved that PR and asked John to review it as well, so maybe just give it another day, in case we can that PR merged first. |
1f292bc to
bb115b6
Compare
|
Viewing the post through GitHub is not rendering the table with the two images correctly (it's showing the markup in-line). I don't know, though, if that needs to be fixed here or if it will still render correctly when viewed through the website. |
|
It renders correctly when built locally. @JStech if that was the only issue, can you approve and merge? |
|
Approved. I can't merge, though. |
felixvd
left a comment
There was a problem hiding this comment.
Didn't build it locally, but it looks good to me. I'm good with merging after the comment is addressed (I don't insist on that change in particular, but just have another look at it).
Co-authored-by: Felix von Drigalski <FvDrigalski@gmail.com>
Description
Please explain the changes you made, including a reference to the related issue if applicable
Checklist
build_locally.shscript