Opened 6 years ago
Closed 5 years ago
#17035 closed enhancement (fixed)
Add an option for indexed generators to display strings without quotes
Reported by: | tscrim | Owned by: | sage-combinat |
---|---|---|---|
Priority: | major | Milestone: | sage-6.10 |
Component: | misc | Keywords: | |
Cc: | sage-combinat, nthiery | Merged in: | |
Authors: | Travis Scrimshaw | Reviewers: | Darij Grinberg |
Report Upstream: | N/A | Work issues: | |
Branch: | 5b546c2 (Commits) | Commit: | 5b546c2b28c52e72785c5e52016321804e249b6b |
Dependencies: | Stopgaps: |
Description
In order to do have the generators print like other variables in Sage. So we'd have the following:
sage: C = CombinatorialFreeModule(QQ, ['x','y','z'], prefix='', bracket=False, string_quotes=False) sage: C.an_element() 2*x + 2*y + 3*z
Thus it matches polynomial rings and other variables in Sage and gives a cleaner output.
Change History (11)
comment:1 Changed 6 years ago by
- Branch set to public/misc/quote_option_indexed_generators-17035
- Commit set to 5b546c2b28c52e72785c5e52016321804e249b6b
- Status changed from new to needs_review
comment:2 follow-up: ↓ 4 Changed 6 years ago by
Shouldn't
sage: M.<a,b,c>=CombinatorialFreeModule(QQ)
work too?
comment:3 Changed 6 years ago by
+1 on the feature, thanks! I am not super convinced by the option name, but did not manage to come up with something better. Ideas anyone?
comment:4 in reply to: ↑ 2 Changed 6 years ago by
Replying to nbruin:
Shouldn't
sage: M.<a,b,c>=CombinatorialFreeModule(QQ)work too?
This definitely would make sense! Given this is not directly related, this can go in a separate ticket at Travis's convenience.
comment:5 Changed 6 years ago by
comment:6 Changed 6 years ago by
- Milestone changed from sage-6.4 to sage-6.8
comment:7 Changed 5 years ago by
- Status changed from needs_review to positive_review
comment:8 Changed 5 years ago by
LGTM!
comment:10 Changed 5 years ago by
- Milestone changed from sage-6.8 to sage-6.10
- Reviewers set to Darij Grinberg
- Status changed from needs_work to positive_review
comment:11 Changed 5 years ago by
- Branch changed from public/misc/quote_option_indexed_generators-17035 to 5b546c2b28c52e72785c5e52016321804e249b6b
- Resolution set to fixed
- Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
New commits:
Another optio for indexed generators.
Fixed failing doctest.