Changes between Initial Version and Version 1 of Ticket #9107, comment 39


Ignore:
Timestamp:
08/22/13 19:15:31 (7 years ago)
Author:
tscrim
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #9107, comment 39

    initial v1  
    11Hey Nicolas and Simon,
    22
    3 The problem comes from the fact that there is a 4 level deep class nesting with a method (which is 5 levels deep) in the `Sets.WithRealizations.ParentMethods.Realizations.super_categories`. I've tried moving this subclass into a separate class, and this solves the pdf build issue but introduces some doctesting errors. I don't think there is a way to extend the nesting level since that is a latex thing, nor do I think we should try since 4 nested classes is a lot IMO. I'm guessing beforehand because of the improper naming, latex did the environments differently...?
     3The problem comes from the fact that there is a 4 level deep class nesting with a method (which is 5 levels deep) in the `Sets.WithRealizations.ParentMethods.Realizations.super_categories`. I've tried moving this subclass into a separate class, and this solves the pdf build issue but introduces some doctesting errors. I don't think there is a to extend the nesting level since that is a latex thing, nor do I think we should try since 4 nested classes is a lot IMO. I'm guessing beforehand because of the improper naming, latex did the environments differently...?
    44
    55Anyways the fix for the pdf build is to remove a level (or two) of class nesting.
     
    77Best,[[BR]]
    88Travis
     9
     10Edit: Here are the errors I get when I move `Sets.WithRealizations` out as a separate class and then assign it into `Sets`:
     11{{{
     12sage -t ../categories/sets_cat.py
     13**********************************************************************
     14File "../categories/sets_cat.py", line 1408, in sage.categories.sets_cat.ParentMethodsForWithRealizations.realizations
     15Failed example:
     16    A.realizations()
     17Expected:
     18    [The subset algebra of {1, 2, 3} over Rational Field in the Fundamental basis, The subset algebra of {1, 2, 3} over Rational Field in the In basis, The subset algebra of {1, 2, 3} over Rational Field in the Out basis]
     19Got:
     20    [The subset algebra of {1, 2, 3} over Rational Field in the Fundamental basis, The subset algebra of {1, 2, 3} over Rational Field in the In basis, The subset algebra of {1, 2, 3} over Rational Field in the Out basis, The subset algebra of {1, 2, 3} over Rational Field in the realization Blah]
     21**********************************************************************
     22File "../categories/sets_cat.py", line 1428, in sage.categories.sets_cat.ParentMethodsForWithRealizations.facade_for
     23Failed example:
     24    A.facade_for()
     25Expected:
     26    [The subset algebra of {1, 2, 3} over Rational Field in the Fundamental basis, The subset algebra of {1, 2, 3} over Rational Field in the In basis, The subset algebra of {1, 2, 3} over Rational Field in the Out basis]
     27Got:
     28    [The subset algebra of {1, 2, 3} over Rational Field in the Fundamental basis, The subset algebra of {1, 2, 3} over Rational Field in the In basis, The subset algebra of {1, 2, 3} over Rational Field in the Out basis, The subset algebra of {1, 2, 3} over Rational Field in the realization Blah]
     29**********************************************************************
     302 items had failures:
     31   1 of   8 in sage.categories.sets_cat.ParentMethodsForWithRealizations.facade_for
     32   1 of   3 in sage.categories.sets_cat.ParentMethodsForWithRealizations.realizations
     33    [241 tests, 2 failures, 0.76 s]
     34----------------------------------------------------------------------
     35sage -t ../categories/sets_cat.py  # 2 doctests failed
     36----------------------------------------------------------------------
     37}}}
     38Any ideas why moving the class out of the nesting doesn't work?