#18427 closed enhancement (fixed)
matroids catalog, optional field of representation
Reported by: | chaoxu | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | sage-6.7 |
Component: | matroid theory | Keywords: | |
Cc: | Stefan | Merged in: | |
Authors: | Chao Xu | Reviewers: | Rudi Pendavingh |
Report Upstream: | N/A | Work issues: | |
Branch: | b1c4de0 (Commits, GitHub, GitLab) | Commit: | |
Dependencies: | Stopgaps: |
Description
Allowing the user to specify an field/ring when creating an matroid.
Example, Wheel would have two new options
def Wheel(n, field=None, ring=None)
matroids.Wheel(4, field=GF(3))
yields a ternary matroid.
Change History (6)
comment:1 Changed 7 years ago by
comment:2 Changed 7 years ago by
- Branch set to u/chaoxu/ticket/18427
comment:3 Changed 7 years ago by
- Commit set to b1c4de0cf3b06cfc4345f612f9afe0d984b16784
- Status changed from new to needs_review
I have made an attempt for the wheel matroids. It returns regular matroid if either no ring/field specified, or the ring is the ring of integers.
comment:4 Changed 7 years ago by
- Reviewers set to Rudi
- Status changed from needs_review to positive_review
Code works, doctests succeed. Positive review.
comment:5 Changed 7 years ago by
- Branch changed from u/chaoxu/ticket/18427 to b1c4de0cf3b06cfc4345f612f9afe0d984b16784
- Resolution set to fixed
- Status changed from positive_review to closed
comment:6 Changed 7 years ago by
- Commit b1c4de0cf3b06cfc4345f612f9afe0d984b16784 deleted
- Reviewers changed from Rudi to Rudi Pendavingh
Note: See
TracTickets for help on using
tickets.
I will be working on this in the coming week.