Opened 7 years ago
Closed 7 years ago
#13411 closed enhancement (fixed)
implement the Holt Graph
Reported by: | chapoton | Owned by: | jason, ncohen, rlm |
---|---|---|---|
Priority: | minor | Milestone: | sage-5.5 |
Component: | graph theory | Keywords: | graph theory |
Cc: | Merged in: | sage-5.5.beta0 | |
Authors: | Frédéric Chapoton | Reviewers: | Nathann Cohen |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
Attachments (2)
Change History (11)
comment:1 Changed 7 years ago by
- Keywords graph theory added
- Status changed from new to needs_review
comment:2 Changed 7 years ago by
comment:3 Changed 7 years ago by
Hellooooooooooooo !!!!
Here is a small patch that attemtps to change that :
sage: map(type,graphs.HoltGraph().vertices()[0]) [<type 'sage.rings.finite_rings.integer_mod.IntegerMod_int'>, <type 'sage.rings.finite_rings.integer_mod.IntegerMod_int'>]
Into the more natural product of integers. In order to avoid very weird bugs later on :-)
It also changes the layout into something slightly different.
Nathann
Changed 7 years ago by
comment:4 Changed 7 years ago by
If you agree with these changes, then this ticket is good to go ! :-)
Nathann
comment:5 Changed 7 years ago by
- Description modified (diff)
- Reviewers set to Nathann Cohen
- Status changed from needs_review to positive_review
ok, and even the bot is happy ! Let's put it in positive review.
comment:6 Changed 7 years ago by
- Description modified (diff)
comment:7 Changed 7 years ago by
- Description modified (diff)
comment:8 Changed 7 years ago by
- Milestone changed from sage-5.4 to sage-5.5
comment:9 Changed 7 years ago by
- Merged in set to sage-5.5.beta0
- Resolution set to fixed
- Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
Damn it.. It seems to depend on something about Cartesian product.