Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Drawing Schlegel diagrams of 4-polytopes using tikz #34113

Open
LaisRast opened this issue Jul 4, 2022 · 9 comments
Open

Drawing Schlegel diagrams of 4-polytopes using tikz #34113

LaisRast opened this issue Jul 4, 2022 · 9 comments

Comments

@LaisRast
Copy link

LaisRast commented Jul 4, 2022

This ticket makes it possible to draw Schlegel diagrams for 4-polytopes.
In particular, it implements the function _tikz_4d_schlegel of the class geometry.polyhedron.plot.Projection and makes it accessible for polytopes.

Example:

sage: C = polytopes.hypercube(4)
sage: Image = C.tikz([-0.0444,-0.6012,-0.7979], 173.25, 2, edge_color="blue", facet_color="black", vertex_color="green")

CC: @slel @jplab @kliem @seblabbe @videlec

Component: geometry

Keywords: tikz, polytope

Author: Laith Rastanawi

Branch/Commit: public/34113 @ bd57a87

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

@LaisRast LaisRast added this to the sage-9.7 milestone Jul 4, 2022
@LaisRast
Copy link
Author

LaisRast commented Jul 4, 2022

Attachment: hypercube.png

@LaisRast
Copy link
Author

LaisRast commented Jul 4, 2022

New commits:

3333167implement _tikz_4d_schlegel
edf9449expose _tikz_4d_schlegel to Projection
bd57a87expose _tikz_4d_schlegel to Polyhedron

@LaisRast
Copy link
Author

LaisRast commented Jul 4, 2022

Branch: public/34113

@LaisRast
Copy link
Author

LaisRast commented Jul 4, 2022

Author: Laith Rastanawi

@LaisRast

This comment has been minimized.

@LaisRast
Copy link
Author

LaisRast commented Jul 4, 2022

Commit: bd57a87

@fchapoton

This comment has been minimized.

@seblabbe
Copy link
Contributor

seblabbe commented Jul 5, 2022

comment:4

Great addition!

Last week, I worked on #33002. I think there will be a conflict between the two tickets. Which ticket should rebase on the other one? Does anybody of you want to review #33002?

@LaisRast
Copy link
Author

LaisRast commented Jul 5, 2022

comment:5

I think this ticket should be rebased on your ticket. Thus I will switch this ticket to needs_work.

@mkoeppe mkoeppe modified the milestones: sage-9.7, sage-9.8 Sep 19, 2022
@mkoeppe mkoeppe removed this from the sage-9.8 milestone Jan 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants