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

Index of functions to hasse_diagram.py #18636

Closed
jm58660 mannequin opened this issue Jun 8, 2015 · 9 comments
Closed

Index of functions to hasse_diagram.py #18636

jm58660 mannequin opened this issue Jun 8, 2015 · 9 comments

Comments

@jm58660
Copy link
Mannequin

jm58660 mannequin commented Jun 8, 2015

Add a alphabetical list of functions to hasse_diagram.py.

CC: @fchapoton

Component: documentation

Author: Jori Mäntysalo

Branch/Commit: u/jmantysalo/index_of_functions_to_hasse_diagram_py @ 8ebdb69

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

@jm58660 jm58660 mannequin added this to the sage-6.8 milestone Jun 8, 2015
@jm58660 jm58660 mannequin added c: documentation labels Jun 8, 2015
@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Jun 8, 2015

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Jun 8, 2015

comment:2

Not so important, but maybe useful addition.


New commits:

8ebdb69Added an index. No code changes.

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Jun 8, 2015

Commit: 8ebdb69

@jm58660 jm58660 mannequin added the s: needs review label Jun 8, 2015
@novoselt
Copy link
Member

novoselt commented Jun 8, 2015

comment:3

Is there a way to have such tables added automatically??? I imagine it is very easy to forget about updating it when functions change...

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Jun 8, 2015

comment:4

Replying to @novoselt:

Is there a way to have such tables added automatically??? I imagine it is very easy to forget about updating it when functions change...

AFAIK no. And adding one should be done on Sphinx, which is used for the documentation.

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Jul 9, 2015

comment:5

Ping. This should be quite trivial to review.

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Jul 17, 2015

comment:6

Frédéric? Just an index added, an easy review.

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Jul 20, 2015

comment:7

Forget. As Nathann made #18926, this is better to implement with it.

(As the "index" is just an alphabetical list. This is totally different with, say, posets.py.)

@jm58660 jm58660 mannequin added s: needs work and removed s: needs review labels Jul 20, 2015
@jm58660 jm58660 mannequin removed this from the sage-6.8 milestone Jul 20, 2015
@jm58660 jm58660 mannequin added the wishlist item label Jul 20, 2015
@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Jul 20, 2015

comment:8

Replying to @jm58660:

Forget. As Nathann made #18926, this is better to implement with it.

...and he already added this index to that ticket. Hence I mark this as duplicate / positive_review ready to close.

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

2 participants