Opened 10 years ago

Closed 10 years ago

#8926 closed defect (fixed)

Family cannot completely handle a class as an argument

Reported by: saliola Owned by: hivert
Priority: major Milestone: sage-4.4.4
Component: combinatorics Keywords:
Cc: sage-combinat Merged in: sage-4.4.4.alpha0
Authors: Florent Hivert Reviewers: Nicolas M. Thiéry
Report Upstream: N/A Work issues: hivert
Branch: Commit:
Dependencies: Stopgaps:

Description 

sage: F = Family(NonNegativeIntegers(), PerfectMatchings)
sage: F
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
...
/home/saliola/Applications/sage-4.4/local/lib/python2.6/site-packages/sage/sets/family.pyc in _repr_(self)
    873             name = name+"(i)"
    874         else:
--> 875             name = self.function.__repr__()
    876             if isinstance(self.function, AttrCallObject):
    877                 name = "i"+name[1:]

TypeError: descriptor '__repr__' of 'sage.structure.sage_object.SageObject' object needs an argument

Attachments (1)

trac_8926_family_repr-fh.patch​ (1.2 KB) - added by hivert 10 years ago.

Download all attachments as: .zip

Change History (5)

Changed 10 years ago by hivert

comment:1 Changed 10 years ago by hivert

  • Authors set to Florent Hivert
  • Status changed from new to needs_review
  • Work issues set to hivert

comment:2 Changed 10 years ago by hivert

  • Reviewers set to Nicolas M. Thiéry
  • Status changed from needs_review to positive_review

From a private e-mail from Nicolas M. Thiéry: 

 - trac_8926_family_repr-fh.patch   # Positive review, assuming test pass

We had a all tests passed on the server massena. Therefore I allow myself to put a positive review on behalf of Nicolas.

comment:3 Changed 10 years ago by hivert

  • Owner changed from sage-combinat to hivert

comment:4 Changed 10 years ago by mhansen

  • Merged in set to sage-4.4.4.alpha0
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.