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.

Sign up for GitHub

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

Jump to bottom

triangulated surface of genus six #34351

Closed
fchapoton opened this issue Aug 12, 2022 · 10 comments
Closed

triangulated surface of genus six #34351

fchapoton opened this issue Aug 12, 2022 · 10 comments
Labels
c: algebraic topology p: minor / 4 t: enhancement
Milestone
sage-9.7

Comments

@fchapoton
Copy link
Contributor

just adding a nice example for simplicial complexes

CC: @slel @dcoudert

Component: algebraic topology

Author: Frédéric Chapoton

Branch/Commit: 1196685

Reviewer: David Coudert

Issue created by migration from https://trac.sagemath.org/ticket/34351
@fchapoton fchapoton added this to the sage-9.7 milestone Aug 12, 2022
@fchapoton
Copy link
Contributor Author

New commits:

8fd2897add an example : nice triangulated surface of genus 6

@fchapoton
Copy link
Contributor Author

Branch: u/chapoton/34351

@fchapoton
Copy link
Contributor Author

Commit: 8fd2897

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 13, 2022

Changed commit from 8fd2897 to 1196685

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 13, 2022

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

1196685tweak the reference

@fchapoton
Copy link
Contributor Author

comment:3

all green lights, so please review

@dcoudert
Copy link
Contributor

Reviewer: David Coudert

@dcoudert
Copy link
Contributor

comment:4

LGTM.

@slel
Copy link
Member

slel commented Aug 13, 2022

comment:5

Another way to define L would be:

    L = ("014 018 023 027 036 049 056 05b 07a 08a 09b 125 126 137 139"
         "147 15a 16b 18b 19a 23b 248 24a 258 269 279 2ab 345 34b 35a"
         "367 389 38a 459 46a 46b 478 568 579 57b 67a 689 78b 9ab"
        ).split()

but I'm fine either way and I agree with the positive review.

@vbraun
Copy link
Member

vbraun commented Aug 30, 2022

Changed branch from u/chapoton/34351 to 1196685

@vbraun vbraun closed this as completed in 59e9feb Aug 30, 2022
@vbraun vbraun mentioned this issue Aug 29, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: algebraic topology p: minor / 4 t: enhancement
Projects
None yet
Milestone
sage-9.7
Development

No branches or pull requests
4 participants
@vbraun @slel @fchapoton @dcoudert