PolyhedralComplex.plot(explosion_factor=1)

[mkoeppe]

CC: @yuan-zhou @jplab

Component: graphics

Author: Matthias Koeppe, Yuan Zhou

Branch/Commit: e8ec0eb

Reviewer: Yuan Zhou, Matthias Koeppe

Issue created by migration from https://trac.sagemath.org/ticket/33596

[mkoeppe]

Branch: u/mkoeppe/polyhedralcomplex_plot_exploded

[mkoeppe]

New commits:

42db852

src/sage/geometry/polyhedral_complex.py (exploded_rainbow_plot, PolyhedralComplex.plot): New

[mkoeppe]

Commit: 42db852

[sagetrac-git]

Changed commit from 42db852 to 1028bb4

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

1028bb4

src/sage/geometry/polyhedral_complex.py: Add documentation

[yuan-zhou]

comment:6

It is a very cool feature!

The plot method should include such an example which is decorated with # optional - sage.plot where the explosion_factor is nonzero.

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

9eebc32

src/sage/geometry/polyhedral_complex.py: Expand example of plot method

[sagetrac-git]

Changed commit from 1028bb4 to 9eebc32

[mkoeppe]

comment:8

Probably not so great that the exploded plot uses rainbow colors implicitly but the non-exploded plot doesn't

[sagetrac-git]

Changed commit from 9eebc32 to b0ed0a7

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

b0ed0a7

src/sage/geometry/polyhedral_complex.py (exploded_rainbow_plot): Accepts dicts for sticky_vertices, sticky_center

[sagetrac-git]

Changed commit from b0ed0a7 to e1d43a9

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

e1d43a9

src/sage/geometry/polyhedral_complex.py: Change to a centrally symmetric example

[yuan-zhou]

comment:11

Can use rainbow color plot when explosion_factor=0 is provided, and use normal plot when explosion_factor is not specified.
Replying to @mkoeppe:

Probably not so great that the exploded plot uses rainbow colors implicitly but the non-exploded plot doesn't

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

3d84635

src/sage/geometry/polyhedral_complex.py (exploded_plot): Rename from exploded_rainbow_plot, handle color='rainbow'

[sagetrac-git]

Changed commit from e1d43a9 to 3d84635

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

85cb04e

src/sage/geometry/polyhedral_complex.py (exploded_plot): Do not set alpha=0.5 by default

[sagetrac-git]

Changed commit from 3d84635 to 85cb04e

[sagetrac-git]

Changed commit from 85cb04e to 568b027

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

568b027

src/sage/geometry/polyhedral_complex.py (exploded_plot): Use same default for point size as Polyhedron.plot

[sagetrac-git]

Changed commit from 568b027 to efc9455

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

efc9455

src/sage/geometry/polyhedral_complex.py (PolyhedralComplex.plot): Handle color='rainbow' in non-exploded plots too

[sagetrac-git]

Changed commit from efc9455 to f0321e5

[mkoeppe]

comment:21

Try now. explosion_factor=0.3 is better in this case

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

cc0a47e

src/sage/geometry/polyhedral_complex.py (exploded_plot): Fix doc of sticky_center

[sagetrac-git]

Changed commit from f0321e5 to cc0a47e

[yuan-zhou]

Changed branch from u/mkoeppe/polyhedralcomplex_plot_exploded to u/yzh/polyhedralcomplex_plot_exploded

[yuan-zhou]

comment:25

Add examples of 3d plots.

---

New commits:

c5a8200

src/sage/geometry/polyhedral_complex.py (plot): Add 3d plot examples

[yuan-zhou]

Changed commit from cc0a47e to c5a8200

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

2b9decf

src/sage/geometry/polyhedral_complex.py (plot): Describe exploded_plot keyword arguments in the docstring

[sagetrac-git]

Changed commit from c5a8200 to 2b9decf

[mkoeppe]

Changed author from Matthias Koeppe to Matthias Koeppe, Yuan Zhou

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

b064146

src/sage/geometry/polyhedral_complex.py (plot): Add 3d plot examples (fixup)

[sagetrac-git]

Changed commit from 2b9decf to b064146

[yuan-zhou]

comment:29

Since points are plotted before polyhedra, will they be covered by the polyhedra? Shall we either use zorder or reverse the plotting order to make vertices visible?

[mkoeppe]

comment:30

good idea to plot them in the reverse order

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

e8ec0eb

src/sage/geometry/polyhedral_complex.py (exploded_plot): Plot polyhedra first, lines next and points last.

[sagetrac-git]

Changed commit from b064146 to e8ec0eb

[yuan-zhou]

Reviewer: Yuan Zhou, Matthias Koeppe

[mkoeppe]

comment:34

Thanks for adding these great examples and for the review

[vbraun]

Changed branch from u/yzh/polyhedralcomplex_plot_exploded to e8ec0eb