Adds Clique Seeding Example #325
Conversation
|
Oops, I just saw I was using networkx to plot the connectivity graphs - which is causing CI to fail. I will fix that. |
…g, and added planning cell to verify that everything works.
…ions which is too slow for a demo notebook
wernerpe
force-pushed
the
clique_seeding_manual_distribution
branch
from
e0a474a to
16dff98
Compare
* Update analytic_grasp test cases * Modify the placement of points for SDF problem * shrink sdf img * lint and removing formatting comments * Update test_analytic_grasp.py * Sync changes --------- Co-authored-by: Nicholas Pfaff <[email protected]>
* increase 5.4b time limit * increase time limit
* modifying grader for new target values * updating notebook * typo fix
* additional avenue of obj files * Simulation Setup Problem Notebook * updated simulation setup notebook * changing wording of 7.1 * updating parser behaviour 7.1 * quick path removal * table top inertia, plant config
* moving notebook to 7 instead of 5 * updating exercise notebook link
homebrew introduced [email protected] and broke our current uses of 3.12.
and a few other minor edits/cleanups.
There was a problem hiding this comment.
Thanks again Pete. I've just pushed some minor fixes to your branch to make it pass CI (using the running_as_notebook variable to guard against things that would be too slow, or actually even loop forever). PTAL.
It still makes the test really slow for CI, so if there is more you can do in this direction, please consider it.
Reviewed 2 of 2 files at r1.
Reviewable status: 0 of 33 files reviewed, 3 unresolved discussions (waiting on @wernerpe)
book/trajectories/iris_builder.ipynb line 1041 at r2 (raw file):
"\n", "\n", "def resize_svg(svg_b, width_user: int):\n",
wow. that's complicated! can't you just call e.g.
# Set the desired width (in inches)
graph.set_size("8,4!")
on the pydot object before calling create_svg?
book/trajectories/iris_builder.ipynb line 1065 at r2 (raw file):
"\n", "# plotting function for later\n", "def plot_region_connectivity(\n",
can we replace/merge this with the VisualizeConnectivity() method that already exists below?
book/data line 1 at r1 (raw file):
Subproject commit 05a36fd9d620241fb93facd4f45e01a7612a7fa7
i've merged the other branch, so let's just make sure we update to that commit.
|
This example is making too many regions for GcsTrajOpt to handle elegantly. Let's wait until the GCS* code lands... and see if it helps. |
I am not sure how to properly add the image to the manipulation-data repo. I opened a separate PR here: RussTedrake/manipulation-data#4
This change is