Opened 12 years ago
Closed 6 years ago
#10261 closed enhancement (fixed)
Improve documentation for word (over str vs over integer)
Reported by: | Sébastien Labbé | Owned by: | Sébastien Labbé |
---|---|---|---|
Priority: | major | Milestone: | sage-6.4 |
Component: | documentation | Keywords: | sd75 |
Cc: | Thierry Monteil | Merged in: | |
Authors: | Sébastien Labbé | Reviewers: | Thierry Monteil |
Report Upstream: | N/A | Work issues: | |
Branch: | 97dada0 (Commits, GitHub, GitLab) | Commit: | 97dada02f2d195de689f870e2db13f2e546e2e75 |
Dependencies: | Stopgaps: |
Description (last modified by )
As noticed by Florent Hivert here, the following two objects :
sage: w = Word('010120') sage: z = Word([0, 1, 0, 1, 2, 0])
have the same string representation :
sage: w word: 010120 sage: z word: 010120
but are not equal :
sage: w == z False
Indeed, w and z are defined on different alphabet :
sage: w[2] '0' sage: z[2] 0
Documentation about this should be improved.
Also that url looks very bad: http://doc.sagemath.org/html/en/reference/combinat/sage/combinat/words/__init__.html
which is referenced from http://doc.sagemath.org/html/en/reference/combinat/index.html
Change History (10)
comment:1 Changed 9 years ago by
Milestone: | sage-5.11 → sage-5.12 |
---|
comment:2 Changed 9 years ago by
Milestone: | sage-6.1 → sage-6.2 |
---|
comment:3 Changed 9 years ago by
Milestone: | sage-6.2 → sage-6.3 |
---|
comment:4 Changed 8 years ago by
Milestone: | sage-6.3 → sage-6.4 |
---|
comment:5 Changed 6 years ago by
Description: | modified (diff) |
---|
comment:6 Changed 6 years ago by
Keywords: | sd75 added |
---|
comment:7 Changed 6 years ago by
Authors: | → Sébastien Labbé |
---|---|
Branch: | → u/slabbe/10261 |
Cc: | Thierry Monteil added |
Commit: | → 4c5e509b2950267684ad14607e5e1a4729024f81 |
Status: | new → needs_review |
comment:8 Changed 6 years ago by
Commit: | 4c5e509b2950267684ad14607e5e1a4729024f81 → 97dada02f2d195de689f870e2db13f2e546e2e75 |
---|
comment:9 Changed 6 years ago by
Reviewers: | → Thierry Monteil |
---|---|
Status: | needs_review → positive_review |
Through this way of indexing the contents of the doc via __init__.py
files is specific to the combinat section (which leads to weird URLs), the commits fix the issue and the doc is improved.
comment:10 Changed 6 years ago by
Branch: | u/slabbe/10261 → 97dada02f2d195de689f870e2db13f2e546e2e75 |
---|---|
Resolution: | → fixed |
Status: | positive_review → closed |
I am cleaning my old trac tickets during Sage Days 75.