Opened 7 years ago
Closed 6 years ago
#13801 closed defect (fixed)
Fix facade set category initialisation
Reported by: | vdelecroix | Owned by: | vdelecroix |
---|---|---|---|
Priority: | major | Milestone: | sage-5.8 |
Component: | categories | Keywords: | category, facade, days45 |
Cc: | sstarosta | Merged in: | sage-5.8.beta0 |
Authors: | Vincent Delecroix | Reviewers: | Travis Scrimshaw |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
Facade sets are used in many places but there are two small errors that prevent from using it everywhere...
The first error concerns multiple categories initialisation. Here is a minimal example
class A(Parent): def __init__(self): Parent.__init__(self, category=((FiniteEnumeratedSets(),Monoids()), facade=True)
for which we get
sage: A() Traceback (most recent call last): ... AttributeError: 'tuple' object has no attribute 'is_subcategory'
The second problem concerns the behavior of .facade_for() which is better explained with a minimal example
class B(Parent): def __init__(self): Parent.__init__(self, category=Sets(), facade=True)
for which we get
sage: b = B() sage: b.facade_for() Traceback (most recent call last): ... AttributeError: 'B_with_category' object has no attribute '_facade_for'
The ticket provides the (trivial) corrections for both problems and add two doctests.
Attachments (1)
Change History (14)
comment:1 Changed 7 years ago by
- Description modified (diff)
- Status changed from new to needs_review
comment:2 follow-up: ↓ 4 Changed 6 years ago by
comment:4 in reply to: ↑ 2 ; follow-up: ↓ 6 Changed 6 years ago by
Replying to slabbe:
Note: you may use ReST syntax for citing trac ticket :
:trac:`12796`
That's now done.
comment:5 Changed 6 years ago by
I modified the behavior of facade_for() in the way it was discussed on combinat-devel (see comment:3 of Travis).
Changed 6 years ago by
comment:6 in reply to: ↑ 4 Changed 6 years ago by
Replying to vdelecroix:
Replying to slabbe:
Note: you may use ReST syntax for citing trac ticket :
:trac:`12796`That's now done.
Actually, I missed one and now it's done!
comment:7 Changed 6 years ago by
- Keywords days45 added
- Reviewers set to Travis Scrimshaw
- Status changed from needs_review to positive_review
Looks good to me now. Thanks Vincent.
Travis
comment:8 Changed 6 years ago by
- Milestone changed from sage-5.7 to sage-5.8
- Resolution set to fixed
- Status changed from positive_review to closed
comment:9 Changed 6 years ago by
Was this merged in 5.7.rc0
?
comment:11 Changed 6 years ago by
- Status changed from new to needs_review
comment:12 Changed 6 years ago by
- Status changed from needs_review to positive_review
comment:13 Changed 6 years ago by
- Merged in set to sage-5.8.beta0
- Resolution set to fixed
- Status changed from positive_review to closed
Note: you may use ReST syntax for citing trac ticket :